Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

Commit 0b97606

Browse files
authored
Merge pull request #625 from openweave/bug/fix_weave_happy_py3_bug
Fix weaave happy py3 bug
2 parents 1fe8c68 + 6dcda89 commit 0b97606

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

.travis/before_install.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ case "${BUILD_TARGET}" in
160160
installdeps "happy-deps"
161161

162162
cd $HOME
163-
git clone --single-branch --branch feature/happy_py3_support https://github.com/openweave/happy.git
163+
git clone https://github.com/openweave/happy.git
164164
cd ${HOME}/happy
165165
make install
166166
python3 pip_packages.py

src/test-apps/happy/lib/WeaveDeviceManager.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -547,27 +547,27 @@ def testWdmClientDataSinkResourceIdentifierMakeResTypeIdBytes(testObject):
547547

548548
def testWdmClientDataSinkSetFlushInvalidInstanceId(testObject):
549549
testObject.createWdmClient()
550-
localeSettingsTrait = testObject.newDataSink(20, 1, "/")
551-
TestCTrait = testObject.newDataSink(593165827, 0, "/")
552-
testObject.setData(localeSettingsTrait, "/1", "en-US")
553-
testObject.setData(TestCTrait, "/1", False)
554-
testObject.setData(TestCTrait, "/2", 15)
555-
testObject.setData(TestCTrait, "/3/1", 16)
556-
testObject.setData(TestCTrait, "/3/2", False)
557-
testObject.setData(TestCTrait, "/4", 17)
550+
localeSettingsTrait = testObject.newDataSink(20, 1, b"/")
551+
TestCTrait = testObject.newDataSink(593165827, 0, b"/")
552+
testObject.setData(localeSettingsTrait, b"/1", b"en-US")
553+
testObject.setData(TestCTrait, b"/1", False)
554+
testObject.setData(TestCTrait, b"/2", 15)
555+
testObject.setData(TestCTrait, b"/3/1", 16)
556+
testObject.setData(TestCTrait, b"/3/2", False)
557+
testObject.setData(TestCTrait, b"/4", 17)
558558
result = testObject.flushUpdate()
559559
if len(result) != 1:
560560
raise ValueError("testWdmClientDataSinkSetFlushInvalidInstanceId fails")
561561
else:
562562
if not (result[0].profileId == 0xb and result[0].statusCode == 0x21):
563563
raise ValueError("testWdmClientDataSinkSetFlushInvalidInstanceId profileId and StatusCode check fails")
564564

565-
print "clear trait: " + str(result[0].dataSink.profileId)
566-
print "clear trait path:" + str(result[0].path)
567-
testObject.deleteData(result[0].dataSink, result[0].path)
565+
print("clear trait: " + str(result[0].dataSink.profileId))
566+
print("clear trait path:" + str(result[0].path))
567+
testObject.deleteData(result[0].dataSink, result[0].path.encode("utf-8"))
568568

569569
testObject.closeWdmClient()
570-
print "testWdmClientDataSinkSetFlushInvalidInstanceId completes"
570+
print("testWdmClientDataSinkSetFlushInvalidInstanceId completes")
571571

572572
def RunWdmClientTest():
573573
print("Run Weave Data Management Test")
@@ -711,7 +711,7 @@ class ExtendedOption (Option):
711711

712712
"""
713713
#Disable unregister since the service automatically removes devices from accounts when they are paired again
714-
print "Register NestService done"
714+
print("Register NestService done")
715715
716716
print ''
717717
print '#################################unregister-real-NestService#################################'
@@ -721,7 +721,7 @@ class ExtendedOption (Option):
721721
print str(ex)
722722
exit()
723723
724-
print "Unregister service done"
724+
print("Unregister service done")
725725
"""
726726
print('')
727727
print('#################################close#################################')

0 commit comments

Comments
 (0)