-
Notifications
You must be signed in to change notification settings - Fork 100
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
chore: Add unit tests to AnyTableLoader #1380
Conversation
✅ Rule acceptance tests passed. |
✅ Rule acceptance tests passed. |
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/TestUtils.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/testgtfs/GtfsStop.java
Outdated
Show resolved
Hide resolved
6422dcd
to
71ef45e
Compare
✅ Rule acceptance tests passed. |
✅ Rule acceptance tests passed. |
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Outdated
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/table/AnyTableLoaderTest.java
Show resolved
Hide resolved
core/src/test/java/org/mobilitydata/gtfsvalidator/validator/ValidatorLoaderTest.java
Outdated
Show resolved
Hide resolved
✅ Rule acceptance tests passed. |
This contribution does not follow the conventions set by the Google Java style guide. Please run the following command line at the root of the project to fix formatting errors: |
…lity verification
Remove DefaultTableHeaderValidator and DefaultValidatorProvider test dependencies
8fb4127
to
dec85a6
Compare
✅ Rule acceptance tests passed. |
✅ Rule acceptance tests passed. |
✅ Rule acceptance tests passed. |
Summary:
Add unit tests to AnyTableLoader preparing the class for future refactoring.
Expected behavior:
Unit tests cover error scenarios and a happy path.
Please make sure these boxes are checked before submitting your pull request - thanks!
gradle test
to make sure you didn't break anything