tailhook / vagga-docker

This is a prototype which brings vagga as the first-class tool to OS X and (possibly) windows through docker's layer of compatibility
10 stars 5 forks source link

Feature/add travis and pytest #13

Closed Arfey closed 7 years ago

Arfey commented 7 years ago

added travis and simple test. For begin just

def test_true():
    assert True

with docker on OS X was problem so naw if [[ "$TRAVIS_OS_NAME" != "osx" ]]

Yet I added Makefile for run tests - make tests

But sometimes raised this error https://travis-ci.org/Arfey/vagga-docker/jobs/268763715#L599-L613

tailhook commented 7 years ago

Hm, yes docker on OS X requires a VM, which doesn't run in another VM where travis runs test. But your approach looks interesting.

But sometimes raised this error https://travis-ci.org/Arfey/vagga-docker/jobs/268763715#L599-L613

This is because of broken mirror, you can fix it with something like this before the test:

echo "alpine-mirror: http://dl-cdn.alpinelinux.org/alpine/" > ~/.vagga.yaml

It's now the default mirror in master branch of vagga, but release is still pending.

codecov-io commented 7 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##             master     #13   +/-   ##
========================================
  Coverage          ?   0.93%           
========================================
  Files             ?      11           
  Lines             ?     214           
  Branches          ?       0           
========================================
  Hits              ?       2           
  Misses            ?     212           
  Partials          ?       0
Impacted Files Coverage Δ
test/first_test.py 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 744d086...9ebd48a. Read the comment docs.

Arfey commented 7 years ago

i will try setting travis in my repo and will create new pull request