Skip to content

airspec: Add test(...) syntax#876

Merged
xerial merged 15 commits intowvlet:masterfrom
xerial:airspec-test
Jan 2, 2020
Merged

airspec: Add test(...) syntax#876
xerial merged 15 commits intowvlet:masterfrom
xerial:airspec-test

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Jan 1, 2020

This will close #872

  • Support local design update
  • Support nesting test cases at ease
  • This test method does not require scalaJsSupport call for Scala.js
  • Update doc

image

image

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 1, 2020

Codecov Report

Merging #876 into master will increase coverage by 0.14%.
The diff coverage is 96.77%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #876      +/-   ##
=========================================
+ Coverage   81.45%   81.6%   +0.14%     
=========================================
  Files         234     235       +1     
  Lines        9131    9197      +66     
  Branches      626     636      +10     
=========================================
+ Hits         7438    7505      +67     
+ Misses       1693    1692       -1
Impacted Files Coverage Δ
.../main/scala/wvlet/airspec/spi/AirSpecContext.scala 100% <ø> (+40%) ⬆️
...pec/src/main/scala/wvlet/airspec/spi/Asserts.scala 66.66% <ø> (ø) ⬆️
...ain/scala/wvlet/airspec/runner/AirSpecLogger.scala 72.72% <100%> (+1.75%) ⬆️
...cala/wvlet/airspec/runner/AirSpecContextImpl.scala 100% <100%> (ø) ⬆️
airspec/src/main/scala/wvlet/airspec/AirSpec.scala 95.65% <100%> (+2.79%) ⬆️
...spec/src/main/scala/wvlet/airspec/AirSpecDef.scala 93.54% <93.54%> (ø)
...scala/wvlet/airspec/runner/AirSpecTaskRunner.scala 89.23% <96.96%> (+3.77%) ⬆️

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 e4f220c...7434293. Read the comment docs.

@xerial xerial changed the title [WIP] airspec: Add test(...) syntax airspec: Add test(...) syntax Jan 2, 2020
@xerial xerial requested review from shimamoto and takezoe January 2, 2020 02:14
@xerial
Copy link
Copy Markdown
Member Author

xerial commented Jan 2, 2020

@shimamoto This new syntax makes writing tests in AirSpec convenient especially if you need to use the same http client instance between tests.

@xerial xerial merged commit 00d8ba9 into wvlet:master Jan 2, 2020
@xerial xerial deleted the airspec-test branch January 2, 2020 14:47
@xerial xerial mentioned this pull request Jan 2, 2020
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.

airspec: Add test(...) syntax for defining local designs

1 participant