Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unit.modules.test_telegram.TelegramModuleTest.test_post_message #52817

Closed
waynew opened this issue May 2, 2019 · 4 comments

Comments

@waynew
Copy link
Contributor

commented May 2, 2019

2019.2.1 failed salt-fedora-29-py3

The dictionary keys returned by TelegramModuleTest.setup_loader_modules() must be an imported module, not <class 'str'>


Traceback (most recent call last):
  File "/tmp/kitchen/testing/tests/support/mixins.py", line 374, in wrapper
    type(module)
RuntimeError: The dictionary keys returned by TelegramModuleTest.setup_loader_modules() must be an imported module, not <class 'str'>
		

This is similar to another error I recently created an issue for. It had "must be an imported module". Ah, it was #52815.

@waynew waynew added Test Failure 2019.2.1 and removed 2019.2.1 labels May 2, 2019
@KChandrashekhar KChandrashekhar added 2019.2.2 and removed 2019.2.1 labels May 3, 2019
@KChandrashekhar

This comment has been minimized.

Copy link

commented May 3, 2019

Not a blocker for 2019.2.1 so we will reconsider fixing this for 2019.2.2 release

@KChandrashekhar KChandrashekhar added this to the Approved milestone May 3, 2019
@Ch3LL

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

re-adding as a blocker

@Ch3LL

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

blocked:

https://jenkinsci.saltstack.com/job/2019.2.1/job/salt-fedora-29-py3/32/console

is not running the tests. I believe this is related to the nox changes recently merged in, but diving into it now.

@Ch3LL

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

bootstrap issue will be fixed by #52968

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.