Skip to content

Commit

Permalink
process_tracker_python-100 Add source/source object character set
Browse files Browse the repository at this point in the history
🐛 Fixed issue where file path for config file was not setting correctly

Issue was found where if the config file was named anything but
'process_tracker_config.ini' it would try to append that to the end.
Have added a new filter to ensure that any file that ends with '.ini'
can be accepted.
  • Loading branch information
Alex Meadows committed Nov 19, 2019
1 parent 37306c8 commit 45c6abe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 0 additions & 2 deletions .travis.yml
Expand Up @@ -15,8 +15,6 @@ before_script:
- mkdir $HOME/.process_tracker
- if [[ ! -z "$DB" ]]; then mv configs/${DB}_config.ini $HOME/.process_tracker/process_tracker_config.ini; fi
- cp configs/process_tracker_config_dev.ini $HOME/.process_tracker/process_tracker_config_dev.ini
- mkdir /tmp/testing
- cp configs/process_tracker_config_dev.ini /tmp/testing/process_tracker_config_dev.ini
- if [[ "$DB" == "postgres" ]]; then psql -f dbscripts/postgresql_account_create.sql -U postgres; fi
- if [[ "$DB" == "postgres" ]]; then psql -f dbscripts/postgresql_process_tracker.sql process_tracker -U postgres; fi
- if [[ "$DB" == "postgres" ]]; then psql -f dbscripts/postgresql_process_tracker_defaults.sql process_tracker -U postgres; fi
Expand Down
4 changes: 2 additions & 2 deletions tests/utilities/test_settings_manager.py
Expand Up @@ -63,10 +63,10 @@ def test_config_location_set_with_ini_file(self):
:return:
"""

expected_result = "/tmp/testing/process_tracker_config_dev.ini"
expected_result = "/home/travis/.process_tracker/process_tracker_config_dev.ini"

given_result = SettingsManager(
config_location="/tmp/testing/process_tracker_config_dev.ini"
config_location="/home/travis/.process_tracker/process_tracker_config_dev.ini"
).config_file

self.assertEqual(expected_result, given_result)
Expand Down

0 comments on commit 45c6abe

Please sign in to comment.