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

Reorder checks in test in case values are being cleared before checked #743

Merged
merged 1 commit into from
Mar 30, 2020

Conversation

estolfo
Copy link
Contributor

@estolfo estolfo commented Mar 26, 2020

An attempt to resolve a reoccurring failure in tests:

ElasticAPM::Instrumenter#end_transaction reports metrics
Failure/Error: expect(spn_self_time.samples[:'span.self_time.sum.us']).to eq 100
NoMethodError: undefined method `samples' for nil:NilClass
# ./spec/elastic_apm/instrumenter_spec.rb:151:in `block in ElasticAPM'

@codecov-io
Copy link

codecov-io commented Mar 26, 2020

Codecov Report

Merging #743 into master will increase coverage by 31.54%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #743       +/-   ##
===========================================
+ Coverage   63.15%   94.69%   +31.54%     
===========================================
  Files         104      108        +4     
  Lines        3148     3395      +247     
===========================================
+ Hits         1988     3215     +1227     
+ Misses       1160      180      -980
Impacted Files Coverage Δ
lib/elastic_apm/spies/mongo.rb 96.96% <0%> (-3.04%) ⬇️
lib/elastic_apm/sql/signature.rb 100% <0%> (ø)
lib/elastic_apm/sql/tokenizer.rb 99.24% <0%> (ø)
lib/elastic_apm/sql/tokens.rb 100% <0%> (ø)
lib/elastic_apm/sql_summarizer.rb 100% <0%> (ø)
lib/elastic_apm/metrics.rb 94.64% <0%> (+3.57%) ⬆️
lib/elastic_apm/metrics/cpu_mem_set.rb 97.95% <0%> (+4.08%) ⬆️
lib/elastic_apm/span/context.rb 100% <0%> (+6.66%) ⬆️
lib/elastic_apm/span.rb 98.14% <0%> (+9.25%) ⬆️
lib/elastic_apm/metrics/set.rb 100% <0%> (+11.53%) ⬆️
... and 74 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44f6e08...57a34a7. Read the comment docs.

@estolfo estolfo merged commit 471ece0 into elastic:master Mar 30, 2020
v1v added a commit to v1v/apm-agent-ruby that referenced this pull request Apr 21, 2020
…release

* upstream/master: (24 commits)
  Ensure that a new central config fetch is scheduled after network error (elastic#772)
  Updates to Resque support and documentation (elastic#768)
  Support recording config option (elastic#765)
  Add tests and adjustments to classes that can be restarted (elastic#766)
  Ensure that the running agent's config is used when restarted (elastic#763)
  The enabled tag is config-enabled
  Add enabled config option replacing active (elastic#762)
  docs: add dynamic badge (elastic#760)
  Config values set to false should be false, not nil (elastic#761)
  Add changelog entry for updating log level on config's logger
  Add changelog entry for logging updated remote config values
  Add changelog entry for dynamic config values
  Check if env variable exists before setting up SimpleCov (elastic#759)
  Log the updated options from central config in addition to cumulative modifications (elastic#758)
  ci(jenkins): merge branch in downstream jobs (elastic#757)
  Log level on logger is updated when Config options are (elastic#755)
  Cobertura coverage (elastic#736)
  Allow Config values to be dynamic (elastic#747)
  Reorder checks in test in case values are being cleared before checked (elastic#743)
  ci(jenkins): disable stages for only docs (elastic#745)
  ...
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