JamesLongman / Squad-Mortar-Calculator-IOS

IOS App: Mortar calculator for the video game Squad
https://itunes.apple.com/app/id1352781413
MIT License
3 stars 0 forks source link

#5 project tests #34

Closed JamesLongman closed 6 years ago

JamesLongman commented 6 years ago

Just testing out the PR, for now, to check that everything is now set up correctly. Apologies for the 32 commits to anyone looking at the commit history, I had quite a bit of difficulty setting the CI solutions up and committing was required to test.

Lots of tests needed so this PR will most likely remain open while work is done over a week or two

codecov-io commented 6 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@58a7799). Click here to learn what that means. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop      #34   +/-   ##
==========================================
  Coverage           ?   27.69%           
==========================================
  Files              ?       32           
  Lines              ?     2026           
  Branches           ?       17           
==========================================
  Hits               ?      561           
  Misses             ?     1465           
  Partials           ?        0
Impacted Files Coverage Δ
Squad Mortar Calculator/calcValues.swift 100% <ø> (ø)
...ulatorUITests/Squad_Mortar_CalculatorUITests.swift 100% <ø> (ø)
Squad Mortar CalculatorTests/extensionsTests.swift 100% <100%> (ø)
...ad Mortar CalculatorTests/calcFunctionsTests.swift 100% <100%> (ø)
...ad Mortar Calculator/PrimaryTabBarController.swift 66.66% <100%> (ø)
Squad Mortar Calculator/calcFunctions.swift 100% <100%> (ø)
Squad Mortar Calculator/extensions.swift 47.29% <100%> (ø)
Squad Mortar CalculatorTests/calcValuesTests.swift 100% <100%> (ø)

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 58a7799...97f2b60. Read the comment docs.

JamesLongman commented 6 years ago

Well that should be it for this PR, will see if checks pass and if so merge