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

v2.2.0 #250

Merged
merged 1 commit into from
Oct 11, 2024
Merged

v2.2.0 #250

merged 1 commit into from
Oct 11, 2024

Conversation

schneems
Copy link
Member

No description provided.

@schneems
Copy link
Member Author

rails_head is failing. It works for me locally

Failure:
TasksTest#test_HTTP_headers [test/integration/tasks_test.rb:130]:
Expected /"Authorization"=>"Basic YWRtaW46c2VjcmV0"/ to match "** Invoke perf:test (first_time)\n** Invoke perf:setup (first_time)\n** Execute perf:setup\n** Invoke perf:rails_load (first_time)\n** Execute perf:rails_load\nDatabase 'db/production.sqlite3' already exists\n** Execute perf:test\nBooting: production\nMethod: GET\nEndpoint: \"foo_secret\"\nHTTP headers: {\"Authorization\" => \"Basic YWRtaW46c2VjcmV0\", \"Cache-Control\" => \"no-cache\"}\n       user     system      total        real\n2 derailed requests  0.024517   0.004175   0.028692 (  0.029228)\n".

bin/rails test /home/runner/work/derailed_benchmarks/derailed_benchmarks/test/integration/tasks_test.rb:121

Test is missing assertions: `test_ips` /home/runner/work/derailed_benchmarks/derailed_benchmarks/test/integration/tasks_test.rb:197
Test is missing assertions: `test_` /home/runner/work/derailed_benchmarks/derailed_benchmarks/test/integration/tasks_test.rb:1[82](https://github.com/zombocom/derailed_benchmarks/actions/runs/11299126977/job/31430656109?pr=250#step:5:83)
Test is missing assertions: `test_objects` /home/runner/work/derailed_benchmarks/derailed_benchmarks/test/integration/tasks_test.rb:185
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>

I think the difference is the whitespace around the => ??

@schneems schneems merged commit 818f5fd into main Oct 11, 2024
65 of 67 checks passed
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.

1 participant