[Github actions] adding test github actions and adds test changes for pytest

[Harveyt47]

Creating test.yml. Still need to add test changes for this to work.

[coveralls]

Coverage decreased (-77.9%) to 16.034% when pulling 48011ac on test-action into 9d0fb5b on master.

[Harveyt47]

@mmcky I still need to add in coveralls

[mmcky]

thanks @Harveyt47 -- if you can give me an update on this sometime that would be great. Thanks.

[Harveyt47]

Hey @mmcky look like I've got coveralls working and pytest passing. I'm not sure why the coverage is so low though

[mmcky]

Perhaps the .coveragerc file is not being used?

https://coverage.readthedocs.io/en/coverage-5.1/config.html

[mmcky]

Can you remind me what this REPO token is for?

Is this for travis to post the coverage results back to the REPO (or something else)?

We may need to rework this part for GitHub Actions.

[mmcky]

@Harveyt47 this may be an issue re: coverage stats.

PyYAML is not installed, skipping .coveralls.yml.
Submitting coverage to coveralls.io...
Coverage submitted!
Job #2e367460bc5316537ebbba8da13cf8f9b92d4ed4.1
https://coveralls.io/jobs/63716041

so the token must be for this service.

• This means we should remove it and add it as a secret.

${{ secrets. COVERALLS_REPO_TOKEN }}

If we don't need .covearalls.yml then we should delete it

[mmcky]

@Harveyt47 it looks like this is running the tests twice due to use of pull_request and push

Best to set this up for pull_request and then a separate on: for push to - branch: master

[mmcky]

thanks @Harveyt47 is there a way to get the Coveralls details as a comment on the PR?

[Harveyt47]

Hi @mmcky I tried changing the Coveralls set up as I believe it thinks it using travis-ci as the build type, as you can see when you click the link in the coveralls comment above. I think that's why the coverage was so low. I have used a new set up which is from the coveralls Github Actions marketplace page here

[mmcky]

@Harveyt47 maybe we should remove the yml file now that #561 has been merged and change this to be the PR to update tests for pytest

[oyamad]

Can this be closed?

[mmcky]

I agree -- thanks @oyamad