-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix adding BMC via CLI #209
Conversation
f5df33d
to
fd79ce3
Compare
c113b10
to
f7ed566
Compare
While starting to test if my changes are working I discovered that much of the data is missing that is required to do so. As such #217 must be fixed before this PR can be worked on. |
77fe6de
to
278d193
Compare
So atm the problem is that when using the production database one cannot load it. I am investigating why this is the case. |
Production database loaded successfully now into my test setup. I will continue testing this PR now manually. |
951a8d7
to
0974631
Compare
Creating the tests is more complex than I thought because somebody thought implementing non-standard authentication was a great idea. As such one more side quest is required to make this work... |
840d664
to
cd6463f
Compare
One last test is failing. After that I need to split/clean up the PR. |
The last failing test is not easy to fix as there are no tests for the CLI that verify that it doesn't break. As such I will need to create them too. This will happen in the context of this PR. |
9742bcf
to
cfcc70c
Compare
0cce94a
to
3236190
Compare
1101997
to
6404450
Compare
07707d0
to
2481380
Compare
4021ed9
to
d4eba3e
Compare
d4eba3e
to
9f96633
Compare
9ba69b6
to
08a732e
Compare
2b1ee4b
to
bcee6d4
Compare
The two tests that are related to the BMC are now passing. As such I believe this PR is improving the code quality. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot test the BMCAPIForm additions, but the changes look correct and make sense.
Fixes #176
Fixes adding the BMC via the CLI.
TODO: