<div dir="ltr">Hi Alvaro,<div>It is not very easy for us to tell unless you share your changes.</div><div>Could you please submit a PR? even if your change is not good for upstream, at least we have a nice setup to review and discuss the code.</div><div><br></div><div>Pierre</div></div><br><div class="gmail_quote"><div dir="ltr">Le mer. 28 déc. 2016 à 00:54, Alvaro Erickson <<a href="mailto:alvaro.erickson@gmail.com">alvaro.erickson@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg">Greetings all,<br class="gmail_msg"><br class="gmail_msg"></div>I have updated the Buildbot source code which I have a fork of. Here are the updates I made:<br class="gmail_msg"><br class="gmail_msg"><ol class="gmail_msg"><li class="gmail_msg">Updated the hgpoller module to look for changes in sub repos<br class="gmail_msg"></li><li class="gmail_msg">Updated the data.changes module</li><li class="gmail_msg">Updated the db.changes module<br class="gmail_msg"></li><li class="gmail_msg">Added 2 columns to the Changes table</li></ol><br class="gmail_msg">And I am currently having problems trying to update the unit tests to verify my code is correct. For example, I added the new column names to the fakedb module. But, I am unsure what the format should be for the variable and its value. Should it be the python variable name and value? Or the database column and value?<br class="gmail_msg"><br class="gmail_msg"></div>So, here is one of my unit test failures:<br class="gmail_msg"><br class="gmail_msg">Traceback (most recent call last):<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/unit/test_www_sse.py", line 43, i<br class="gmail_msg">n test_listen<br class="gmail_msg">    self.assertReceivesChangeNewMessage(self.request)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/unit/test_www_sse.py", line 110, <br class="gmail_msg">in assertReceivesChangeNewMessage<br class="gmail_msg">    ("changes", "500", "new"), test_data_changes.Change.changeEvent)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/fake/fakemq.py", line 58, in call<br class="gmail_msg">Consumer<br class="gmail_msg">    validation.verifyMessage(self.testcase, routingKey, msg)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/util/validation.py", line 657, in<br class="gmail_msg"> verifyMessage<br class="gmail_msg">    (routingKey, (routingKey, message_)))<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/util/validation.py", line 644, in<br class="gmail_msg"> _verify<br class="gmail_msg">    testcase.fail(msg)<br class="gmail_msg">twisted.trial.unittest.FailTest: changes message has unexpected keys '<new Changes column 1>', '<new Changes column 2>'<br class="gmail_msg"><br class="gmail_msg">buildbot.test.unit.test_www_sse.EventResource.test_listen<br class="gmail_msg"><br class="gmail_msg"></div>So, where do I add the new expected columns for the buildbot.test.unit.test_www_sse.EventResource.test_listen to find?<br class="gmail_msg"><br class="gmail_msg"></div>Finally, here is an error I have:<br class="gmail_msg"><br class="gmail_msg">Traceback (most recent call last):<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/sandbox/local/lib/python2.7/site-packages/twisted/inte<br class="gmail_msg">rnet/defer.py", line 1299, in _inlineCallbacks<br class="gmail_msg">    result = g.send(result)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/changes/hgpoller.py", line 428, in _pr<br class="gmail_msg">ocessChanges<br class="gmail_msg">    errortoo=False)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/util/gpo.py", line 78, in patched<br class="gmail_msg">_getProcessOutput<br class="gmail_msg">    path=path)<br class="gmail_msg">  File "/srv/home/bobthebuilder/buildbothost/repositories/buildbot/master/buildbot/test/util/gpo.py", line 98, in patched<br class="gmail_msg">_getProcessOutputAndValue<br class="gmail_msg">    % (bin, args))<br class="gmail_msg">twisted.trial.unittest.FailTest: got command hg [<hg command>] when no further commands were expected<br class="gmail_msg"><br class="gmail_msg">buildbot.test.unit.test_changes_hgpoller.HgPollerNoTimestamp.test_poll_initial<br class="gmail_msg">buildbot.test.unit.test_changes_hgpoller.TestHgPoller.test_poll_initial<br class="gmail_msg"><br class="gmail_msg"></div>In the hgpoller module, where should I put the my call to check for further changes? Or the best way to solve the error?<br class="gmail_msg"><br class="gmail_msg"></div>Regards,<br class="gmail_msg"><br class="gmail_msg"></div>Alvaro<br class="gmail_msg"></div>
_______________________________________________<br class="gmail_msg">
users mailing list<br class="gmail_msg">
<a href="mailto:users@buildbot.net" class="gmail_msg" target="_blank">users@buildbot.net</a><br class="gmail_msg">
<a href="https://lists.buildbot.net/mailman/listinfo/users" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.buildbot.net/mailman/listinfo/users</a></blockquote></div>