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 with rebar3 #234

Merged
merged 16 commits into from Apr 1, 2020
Merged

Test with rebar3 #234

merged 16 commits into from Apr 1, 2020

Conversation

kostis
Copy link
Collaborator

@kostis kostis commented Apr 1, 2020

PropEr has been running its test suite with a home-brewed patched version of rebar for very a long time now, because its test suite was full of compiler warnings when compiled with the official rebar and/or with rebar3.

This PR finally(!) cleans up the PropEr test suite from all the warnings and removes the patched version of rebar from the repository. From now on, tests are run with rebar3.

kostis added 10 commits April 1, 2020 14:14
WIP: Let's see how many warnings this is generating...
PR proper-testing#184, along with various improvements and very good tests, also
introduced a test that does not make much sense: containing a ?SUCHTHAT
as the third argument of a ?LET macro.  This commit removes that test.

See also the comment and discussion at:
proper-testing@6540152#commitcomment-35664733
and Thanks for All the Fish
@kostis kostis added this to the Update .travis.yml to properly measure coverage milestone Apr 1, 2020
@codecov-io
Copy link

codecov-io commented Apr 1, 2020

Codecov Report

Merging #234 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
- Coverage   83.05%   83.02%   -0.04%     
==========================================
  Files          13       13              
  Lines        3169     3169              
==========================================
- Hits         2632     2631       -1     
- Misses        537      538       +1     
Impacted Files Coverage Δ
src/proper_types.erl 91.77% <ø> (ø)
src/proper_arith.erl 85.43% <0.00%> (-0.98%) ⬇️
src/proper_shrink.erl 95.21% <0.00%> (-0.54%) ⬇️
src/proper.erl 84.88% <0.00%> (+0.15%) ⬆️

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 6c5dc68...74ed3fa. Read the comment docs.

@kostis kostis removed this from the Update .travis.yml to properly measure coverage milestone Apr 1, 2020
@kostis kostis merged commit 0365dd2 into proper-testing:master Apr 1, 2020
@kostis kostis deleted the test-with-rebar3 branch April 1, 2020 21:37
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.

None yet

2 participants