mike820324 / microProxy

A http/https interceptor written in pure python.
MIT License
17 stars 3 forks source link

[WIP]: Refine Unittest #135 #148

Closed mike820324 closed 8 years ago

mike820324 commented 8 years ago

This is a tracking PR for unittest update.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-12.7%) to 44.401% when pulling f6c78fdf32adb54e7572eacf8b6d9488f9cb75ed on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-12.7%) to 44.401% when pulling 7c1c1063c1b7dbae5afae4d39ae0b487c68c3ac2 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-12.1%) to 44.966% when pulling 70e321635de9bb694fc7b49eb4d1d70b30d80413 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-6.9%) to 50.23% when pulling 58599d9fb9291af99a6e279bedbb6546df80a7b6 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

mike820324 commented 8 years ago

Add MicroProxyIOStreamTest.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-6.9%) to 50.201% when pulling d9c21b9ffd00c8529c6f77be126a32b278059db2 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

mike820324 commented 8 years ago

Add start_tls test for MicroProxyIOStream.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-3.4%) to 53.709% when pulling 53594429b4dd9ffbf3f02de40c1afd6538ef9803 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.9%) to 55.25% when pulling b7ab150d704404cddbf366d05590e68ee4c621a1 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

mike820324 commented 8 years ago

Almost the same as before.

mike820324 commented 8 years ago

Add command line options check just like mitmproxy did. So that we won't by accident change the command line interface.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.8%) to 55.326% when pulling 5b52b713f7d978fc6067fe55a1a7a5da02921b56 on unittest-#135 into c8d32ca910564838cdf089625cb8c57c4f8405aa on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.8%) to 57.84% when pulling f930c48fe7f8b9c60d15e98cbf7da064decbfca4 on unittest-#135 into 20c47a112d90dfe9364d142f5ed0e1b8ca9672d5 on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.0%) to 58.063% when pulling 073ed4e18a78416193db36386bc711d929749c33 on unittest-#135 into 20c47a112d90dfe9364d142f5ed0e1b8ca9672d5 on master.

chhsiao90 commented 8 years ago

I found that more open source project that use the pattern: TestCamelCase for class name than CamelCaseTest. So I rename every unittest class for match the pattern. And we would follow this pattern in the future.

chhsiao90 commented 8 years ago

Please help rebase it and I will pull it after you done! haha

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.0%) to 58.063% when pulling 073ed4e18a78416193db36386bc711d929749c33 on unittest-#135 into 20c47a112d90dfe9364d142f5ed0e1b8ca9672d5 on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.0%) to 58.063% when pulling 50b1e7a47b52c99d6c7463841e15d7dc9558fc67 on unittest-#135 into a02dbafc45d88e4afda3a14ce03544362a8932f7 on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+2.5%) to 59.604% when pulling ce1760f4b7ec6031f4b6047f9ade4278b4ea89ca on unittest-#135 into a02dbafc45d88e4afda3a14ce03544362a8932f7 on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+3.0%) to 60.07% when pulling 4959dbac5f684cae5de1cfd9de7dcbded657c2a3 on unittest-#135 into a02dbafc45d88e4afda3a14ce03544362a8932f7 on master.

chhsiao90 commented 8 years ago

@mike820324 , do you have local modification on this branch? Let me do the rebase because I'm afraid there is conflict in it!

mike820324 commented 8 years ago

Please rebase, I'll stash the local modification.

mike820324 commented 8 years ago

When you finished, ping me, tks.

chhsiao90 commented 8 years ago

@mike820324 , rebase done!

coveralls commented 8 years ago

Coverage Status

Coverage increased (+2.8%) to 60.376% when pulling 5a06424032a703582119a19fe563df43cc2ba50f on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

mike820324 commented 8 years ago

Thanks, will add new test up.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+2.8%) to 60.376% when pulling f2a2d9e7ee49c16d693cf4395be18c101f4108d7 on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+4.0%) to 61.538% when pulling 66f6f2472803306f94947af7267f4f5159c5348a on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

mike820324 commented 8 years ago

Update socks layer test.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+4.0%) to 61.56% when pulling 69620649801b2ab0915adb8ae4ada40540aaf1bd on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

mike820324 commented 8 years ago

Rebase needed.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+5.07%) to 62.64% when pulling 294b0ad0d73d733e3c9fbbadd1c39ce478844f2f on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+5.4%) to 62.936% when pulling e70982d9051546873fb7440b990bb2ff5b346a54 on unittest-#135 into 8c2a6c6169d830650e1e37cbdeb7b965c908dc8a on master.

mike820324 commented 8 years ago

@chhsiao90 when the following checklist is done, I would like to issue a first merge. Currently, this PR is become too big that when some other issue merge to master, it would be very hard to rebase.

chhsiao90 commented 8 years ago

Agreed!

coveralls commented 8 years ago

Coverage Status

Coverage increased (+5.8%) to 63.324% when pulling 04d674c0fbcc438f3614c6568453bc72c1546706 on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+5.8%) to 63.324% when pulling 9f60f708c1c3eea86e12ae98b24eba226bb63137 on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+9.0%) to 66.479% when pulling 592177e07d8def2613e1fbec558b45d42d4f59fc on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+10.7%) to 68.215% when pulling e35bc1ef23b27b672e6ea63c28087c93926d1c98 on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+13.6%) to 71.122% when pulling fb4767c605e7abb2bdbd429e4afd3ac7a8721a30 on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

chhsiao90 commented 8 years ago

@mike820324 my part done for http1/http2/replay !

mike820324 commented 8 years ago

I'll finish the rest of the test on the weekend.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+13.6%) to 71.122% when pulling 8d7ae69b118c4b09742fda571efd3fcdfb7af45a on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+14.2%) to 71.703% when pulling 54c4646ed9e557e2aa678730934feca99cbf1b61 on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+14.2%) to 71.703% when pulling 8843a3f345f47d762ab2b45cbedf3679b12948db on unittest-#135 into 80c9aba4e298814f10abb0aa6a9464b139e7a60e on master.