Resolved #147: Add Server state and related API.

[mike820324]

• Add ServerContext
• Add init_server_state and get_server_state
• Add server_state parameter when initialize application.layer instance.
• Update unittest.
• Some code cleanup.

[mike820324]

@chhsiao90 Help me review thanks.

[mike820324]

I will update unittest for server state later.

[coveralls]

Coverage decreased (-0.3%) to 79.241% when pulling 69ccc04c22a09e4e795bb92a0264a7d102c87819 on server-state-#147 into 6ea6b13ebe586ea9268971d0f620ad6e6e5b3105 on master.

[chhsiao90]

Great work! LGTM~

Had some suggestion about the global state of server_state. (Global state is hard to maintain!) How I think to remove the global state of server_state

• Remove the global state in server_state.py (init_server_state just return ServerContext, don't keep the instance) CodeReference
• Init server_state instance and pass ServerContext to ProxyServer in proxy.py as its initialize field CodeReference
• Call run_layers with the server_state instance in ProxyServer CodeReference

And the layer_manager in ProxyServer I think we could remove self.layer_manager = layer_manager in __init__ method. Because layer_manager had changed from object to a module, it's not object anymore, so I think there is no need to assign it to ProxyServer as its field.

How do you think :)

[coveralls]

Coverage decreased (-2.4%) to 77.06% when pulling 464a96000973502484782cfc71ec931aa7fff632 on server-state-#147 into 6ea6b13ebe586ea9268971d0f620ad6e6e5b3105 on master.

[mike820324]

@chhsiao90 I have modified the code based on the previous review.

[mike820324]

Thanks. You have used the new github feature. XD

[chhsiao90]

Yeah, it's cool~ XD

[chhsiao90]

There is no reject options, so sad...

[coveralls]

Coverage increased (+0.4%) to 79.858% when pulling 6ee31163c05eee3f5cff8f1c2d8b1d3fbb3420d9 on server-state-#147 into 6ea6b13ebe586ea9268971d0f620ad6e6e5b3105 on master.

[mike820324]

Don't reject me\ QAQ/