Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Include unit tests for data processing #56

Open
wants to merge 19 commits into
base: staging
Choose a base branch
from

Conversation

D3vZro
Copy link
Collaborator

@D3vZro D3vZro commented Feb 17, 2025

Closes #4

Checklist

Formalities:

  • PR targets staging, if compare branch is chore/* or feat/*
  • PR title is formatted as a commit
  • New functions are commented in the source file
  • Notes are added to wiki

Quality assurance:

  • Manual build and unit/component testing:
    • Linux
    • macOS
    • Windows
  • Manual End-To-End testing:
    • Upload of query batch
    • Computation of routing results
    • Download plan
    • Upload plan
    • Compare functionality

For staging -> master

  • Check for relevant open issues
  • After merge: Create release

@D3vZro D3vZro added the Data Processing Data Processing Team label Feb 17, 2025
@D3vZro D3vZro added this to the Iteration 5 milestone Feb 17, 2025
@D3vZro D3vZro self-assigned this Feb 17, 2025
Kvnstrck and others added 8 commits February 17, 2025 22:14
…50)

* feat: introduce new compareItinerary method for excluding attributes

* chore: add comments and documentation
fix: fix cssClasses when downloading
…from motis/ui (#52)

* feat: implementation of ConnectionDetail.svelte and helper files

* feat: minor style changes & language removal & checking for undefined Itineraries in itinerary ov altered

- removed unnecessary languages like de, pl, fr and more
- scaled down logo by a bit
- checking for undefined itineraries in itinerary ov now happens through checking length and amount of legs

* chore: reordering of files from lib into ui folder

* chore: remove old "Leg" class

* chore: add documentation for helper methods / components
* feat: independent view of itineraries

* fix: swap itinerary and default itinerary overview labels
@D3vZro D3vZro linked an issue Feb 17, 2025 that may be closed by this pull request
@D3vZro D3vZro marked this pull request as ready for review February 18, 2025 11:19
Copy link

@Kha1r0 Kha1r0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

npm run test output on macOS 15.3.1

npm run test output.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Data Processing Data Processing Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set up testing environment
4 participants