Skip to content
This repository has been archived by the owner on Nov 21, 2019. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.

run unit / functional tests #43

Open
lifehackjim opened this issue Apr 13, 2018 · 2 comments
Open

run unit / functional tests #43

lifehackjim opened this issue Apr 13, 2018 · 2 comments
Assignees
Labels
Milestone

Comments

@lifehackjim
Copy link
Contributor

fix any errors!!

@lifehackjim lifehackjim added this to the 2.2.3 milestone Apr 13, 2018
@lifehackjim lifehackjim self-assigned this Apr 13, 2018
lifehackjim pushed a commit that referenced this issue Apr 15, 2018
Various tests for pytan.utils.dehumanize.* had static comparison lists which needed to be updated to always expect 'params' to be returned in each dictionary returned in the list by dehumanize.*
@lifehackjim
Copy link
Contributor Author

unit tests validated!

lifehackjim pushed a commit that referenced this issue Apr 15, 2018
lifehackjim pushed a commit that referenced this issue Apr 15, 2018
lifehackjim pushed a commit that referenced this issue Apr 15, 2018
requests.get() against threaded_http on https://localhost:4433 now
returns:

"bad handshake: Error([('SSL routines', 'ssl3_get_record', 'wrong
version number')],)"

instead of an error that ends with "unknown protocol".

Changed expected error string to just be .*
@lifehackjim
Copy link
Contributor Author

add fixtures to setup class for valid server tests

  • create a package with parameters
  • create a package without parameters
  • create a package that uses sensor sourced parameters
  • get a question for use in tests (esp create object)
  • get rid of custom tags references
  • deploy action should be use new fixtures

lifehackjim pushed a commit that referenced this issue Apr 15, 2018
lifehackjim pushed a commit that referenced this issue Apr 15, 2018
Now verifies parameters before checking run == True/False.  This allows
parameter check to happen earlier. correlated to #43
lifehackjim pushed a commit that referenced this issue Apr 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant