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 house-of-councillor: reduce checking #213

Merged

Conversation

tikkss
Copy link
Contributor

@tikkss tikkss commented Sep 4, 2024

GitHub: GH-188

Because csv file is too big (9,462 rows and 39 columns).

Before this change:

$ time ruby test/run-test.rb -t HouseOfCouncillorTest:::bill --verbose=important-only
Finished in 2.352206 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m2.953s
user    0m2.701s
sys     0m0.153s

After this change:

$ time ruby test/run-test.rb -t HouseOfCouncillorTest:::bill --verbose=important-only
Finished in 0.002077 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.566s
user    0m0.382s
sys     0m0.092s

GitHub: red-data-toolsGH-188

Because csv file is too big (9,462 rows and 39 columns).

Before this change:

```console
$ time ruby test/run-test.rb -t HouseOfCouncillorTest:::bill --verbose=important-only
Finished in 2.352206 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m2.953s
user    0m2.701s
sys     0m0.153s
```

After this change:

```console
$ time ruby test/run-test.rb -t HouseOfCouncillorTest:::bill --verbose=important-only
Finished in 0.002077 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.566s
user    0m0.382s
sys     0m0.092s
```
@kou kou merged commit f8c42e3 into red-data-tools:master Sep 5, 2024
12 checks passed
@kou
Copy link
Member

kou commented Sep 5, 2024

Thanks.

@tikkss tikkss deleted the test-house-of-councillor-reduce-checking branch September 5, 2024 03:07
@tikkss
Copy link
Contributor Author

tikkss commented Sep 5, 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