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

test geolonia: reduce checking #202

Merged
merged 1 commit into from
Aug 31, 2024

Conversation

tikkss
Copy link
Contributor

@tikkss tikkss commented Aug 30, 2024

GitHub: GH-188

Because csv file is too big (277,656 rows).

Before this change:

$ time ruby test/run-test.rb -t GeoloniaTest --verbose=important-only
Finished in 5.998956 seconds.
2 tests, 2 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m6.603s
user    0m6.146s
sys     0m0.346s

After this change:

$ time ruby test/run-test.rb -t GeoloniaTest --verbose=important-only
Finished in 0.001634 seconds.
2 tests, 2 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.564s
user    0m0.388s
sys     0m0.087s

GitHub: red-data-toolsGH-188

Because csv file is too big (277,656 rows).

Before this change:

```console
$ time ruby test/run-test.rb -t GeoloniaTest --verbose=important-only
Finished in 5.998956 seconds.
2 tests, 2 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m6.603s
user    0m6.146s
sys     0m0.346s
```

After this change:

```console
$ time ruby test/run-test.rb -t GeoloniaTest --verbose=important-only
Finished in 0.001634 seconds.
2 tests, 2 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.564s
user    0m0.388s
sys     0m0.087s
```
@kou kou merged commit 01295a1 into red-data-tools:master Aug 31, 2024
0 of 12 checks passed
@kou
Copy link
Member

kou commented Aug 31, 2024

Thanks.

@tikkss tikkss deleted the test-geolonia-reduce-checking branch August 31, 2024 10:56
@tikkss
Copy link
Contributor Author

tikkss commented Aug 31, 2024

Thanks for your review!

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

Successfully merging this pull request may close these issues.

2 participants