search

Duke-GCB / bespin-cli

Command line client for Bespin
MIT License
0 stars 0 forks source link

fix test that fails with PyYAML==5.1b5 #31

Closed johnbradley closed 5 years ago

johnbradley commented 5 years ago

Deploy failed due to newer pyyaml installed in circleci: https://circleci.com/gh/Duke-GCB/bespin-cli/116

Error:

FAIL: test_write_config (bespin.test_config.ConfigFileTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/circleci/project/bespin/test_config.py", line 53, in test_write_config
    self.assertEqual('{token: Secret, url: someurl}', write_strs.strip())
AssertionError: '{token: Secret, url: someurl}' != 'token: Secret\nurl: someurl'

Test is kind of fragile and expects the content to look like JSON(which is allowable in YAML). https://github.com/Duke-GCB/bespin-cli/blob/f3b1871011f414e1769e6ec7914aabf484fabcf8/bespin/test_config.py#L53