About testLocalDiscoveryToggle and testPrinterOffSendGoodbyePacket about cloudprint_logocert HOT 7 CLOSED

For testLocalDiscoveryToggle, the assumption is that the listener will not discover any mdns announcements once Local Discovery is turned off. I see there was a change from using self.browser to mdns_browser a couple of months ago. I wonder if that could have affected this test.

Could you do me a favor and compare the source code of the old and new LogoCert Tool and see if there are any differences that look like they might have affected these two tests?

from cloudprint_logocert.

I have compared the source code of the old and new LogoCert Tool.
Here is the difference of the _mdns.py file which might be the reason why these two issues occurs.

Old Tool

def add_service(self, zeroconf, type, name):
self.logger.info("Service added: '%s' (type is %s)", name, type)
self.discovered[name] = True

New Tool

def add_service(self, zeroconf, service_type, name):
self.logger.info('Service added: "%s" (type is %s)', name, service_type)
self.discovered[name] = {}
self.discovered[name]['proto'] = service_type
self.discovered[name]['found'] = True

I think there is something wrong with the testcert.py of the new tool in line 1841 and line 1929(code is same as line 1841).
So I tried to modify the source as below.
before: self.assertFalse(mdns_browser.listener.discovered[k])
after : self.assertFalse(mdns_browser.listener.discovered[k]['found'])
It should be also necessary to remove the code of line 1818 where compile error would be happened here. I commented it.
With the two modifications, fortunately the two test results changed to "Passed".
Please confirm if my modifications are right.

There is a new issue I want to confirm which occurs just because the order of the tests in my opinion.
When I run testLocalDiscoveryToggle and testPrinterOffSendGoodbyePacket separately, there is no problem. But after running testPrinterIdleNoBroadcastPrivet, both of testLocalDiscoveryToggle and testPrinterOffSendGoodbyePacket will be failed.
It seems like the execution of testPrinterIdleNoBroadcastPrivet would affect the following test.

from cloudprint_logocert.

Yes, looks like your logic might be correct. I'll take a look at the source
code this week and get a change submitted.

Kelly
kdLucas

On Fri, Apr 1, 2016 at 3:30 AM, YYWeave [email protected] wrote:

I have compared the source code of the old and new LogoCert Tool.
Here is the difference of the _mdns.py file which might be the reason why
these two issues occurs.
Old Tool

def add_service(self, zeroconf, type, name):
self.logger.info("Service added: '%s' (type is %s)", name, type)
self.discovered[name] = True
New Tool

def add_service(self, zeroconf, service_type, name):
self.logger.info('Service added: "%s" (type is %s)', name, service_type)
self.discovered[name] = {}
self.discovered[name]['proto'] = service_type
self.discovered[name]['found'] = True

I think there is something wrong with the testcert.py of the new tool in
line 1841 and line 1929(code is same as line 1841).
So I tried to modify the source as below.
before: self.assertFalse(mdns_browser.listener.discovered[k])
after : self.assertFalse(mdns_browser.listener.discovered[k]['found'])
It should be also necessary to remove the code of line 1818 where compile
error would be happened here. I commented it.
With the two modifications, fortunately the two test results changed to
"Passed".
Please confirm if my modifications are right.

There is a new issue I want to confirm which occurs just because the order
of the tests in my opinion.
When I run testLocalDiscoveryToggle and testPrinterOffSendGoodbyePacket
separately, there is no problem. But after running
testPrinterIdleNoBroadcastPrivet, both of testLocalDiscoveryToggle and
testPrinterOffSendGoodbyePacket will be failed.
It seems like the execution of testPrinterIdleNoBroadcastPrivet would
affect the following test.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#21 (comment)

from cloudprint_logocert.

I made some changes, see if this fixes the issues.

I don't think the testPrinterIdleNoBroadcastPrivet should affect any tests,
as it's just listening to see if any Privet packets were advertised over a
given period of time.

Kelly
kdLucas

On Fri, Apr 1, 2016 at 3:30 AM, YYWeave [email protected] wrote:

I have compared the source code of the old and new LogoCert Tool.
Here is the difference of the _mdns.py file which might be the reason why
these two issues occurs.
Old Tool

def add_service(self, zeroconf, type, name):
self.logger.info("Service added: '%s' (type is %s)", name, type)
self.discovered[name] = True
New Tool

def add_service(self, zeroconf, service_type, name):
self.logger.info('Service added: "%s" (type is %s)', name, service_type)
self.discovered[name] = {}
self.discovered[name]['proto'] = service_type
self.discovered[name]['found'] = True

I think there is something wrong with the testcert.py of the new tool in
line 1841 and line 1929(code is same as line 1841).
So I tried to modify the source as below.
before: self.assertFalse(mdns_browser.listener.discovered[k])
after : self.assertFalse(mdns_browser.listener.discovered[k]['found'])
It should be also necessary to remove the code of line 1818 where compile
error would be happened here. I commented it.
With the two modifications, fortunately the two test results changed to
"Passed".
Please confirm if my modifications are right.

There is a new issue I want to confirm which occurs just because the order
of the tests in my opinion.
When I run testLocalDiscoveryToggle and testPrinterOffSendGoodbyePacket
separately, there is no problem. But after running
testPrinterIdleNoBroadcastPrivet, both of testLocalDiscoveryToggle and
testPrinterOffSendGoodbyePacket will be failed.
It seems like the execution of testPrinterIdleNoBroadcastPrivet would
affect the following test.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#21 (comment)

from cloudprint_logocert.

Please verify if the changes fixed this.

from cloudprint_logocert.

Only the issue of testLocalDiscoveryToggle was fixed.
I think the change in line1841 of testcert.py should be also modified in line1865, line1900 and line1929 as the same reason. I tried it and testPrinterOffSendGoodbyePacket was fixed.
Please confirm if I am right.

from cloudprint_logocert.

Ok, I've committed the changes, please verify the fix.

from cloudprint_logocert.