Commits · f2d36e8e9e999d926c98584f81c4d4f177c0389f · nexedi / neoppod

23 Jul, 2009 30 commits

Move master/connector.py content into tests/__init__.py to be consistent with · f2d36e8e

Grégory Wisniewski authored Jul 23, 2009

the base test class placement.
Fix imports in test files for this modification the NeoTestBase class.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@980 71dcc9de-d417-0410-9af5-da40c76e7ee4

f2d36e8e

Tests runner import each test file separately. · a13a11ba

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@979 71dcc9de-d417-0410-9af5-da40c76e7ee4

a13a11ba

Don't import test cases in module's __init__ to avoid crash all tests with an · 88dafac5
Grégory Wisniewski authored Jul 23, 2009
```
import error.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@978 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
88dafac5

Move all tests under neo.tests namespace. · 446b4898

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@977 71dcc9de-d417-0410-9af5-da40c76e7ee4

446b4898

Rename tests runner as 'runner', in one commit to create a directory with the · e90f834c
Grégory Wisniewski authored Jul 23, 2009
```
same name.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@976 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
e90f834c

Grégory Wisniewski authored Jul 23, 2009

no conflict can happen (known by address but not UUID), because the recovery is
done at startup an no storage nodes are known by address.

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@975 71dcc9de-d417-0410-9af5-da40c76e7ee4

8940d047

Remove TODO, master ask for nodes and partition table itself. · e141d8b4
Grégory Wisniewski authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@974 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
e141d8b4

Fix imports in unit tests · ab39302c

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@973 71dcc9de-d417-0410-9af5-da40c76e7ee4

ab39302c

Disable "Method could be a function" warning. · 41889d6a

Vincent Pelletier authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@972 71dcc9de-d417-0410-9af5-da40c76e7ee4

41889d6a

Soften the maximum number of argument limit. · 38257e83

Vincent Pelletier authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@971 71dcc9de-d417-0410-9af5-da40c76e7ee4

38257e83

Fix variable access. · 2b2950a8

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@970 71dcc9de-d417-0410-9af5-da40c76e7ee4

2b2950a8

Allow functions to start with more than one underscore (and update methods too). · 2d197e51

Vincent Pelletier authored Jul 23, 2009

Simplify method regex.
Allow method names to be used as local variables.
Allow any naming at global level (to be restricted later, when protocol.py content will be accepted by pylint and when decorators will be removed).

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@969 71dcc9de-d417-0410-9af5-da40c76e7ee4

2d197e51

Fix naming convnetion of local variable. Found by pylint. · b81e3477

Vincent Pelletier authored Jul 23, 2009

Fix shadowing "callable" global.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@968 71dcc9de-d417-0410-9af5-da40c76e7ee4

b81e3477

Follow naming conventions for class names in epoll.py. Found by pylint. · 88a4d5b9
Vincent Pelletier authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@967 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
88a4d5b9

It is not allowed, per coding style rules, to start a local variable name with... · 1fb845ad

Vincent Pelletier authored Jul 23, 2009

It is not allowed, per coding style rules, to start a local variable name with an underscore. Found by pylint.
Fixes a bug introduced in r958 (variable renaming forgotten on "if" line).

git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@966 71dcc9de-d417-0410-9af5-da40c76e7ee4

1fb845ad

Use is* methods from Node class, suppress some imports by using protocol.*. · 64215aef
Grégory Wisniewski authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@965 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
64215aef
Fix first parameter of "super" function. Found by pylint. · 62e025e3
Vincent Pelletier authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@964 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
62e025e3

Remove this check, implied by current handler. · 76a10f89

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@963 71dcc9de-d417-0410-9af5-da40c76e7ee4

76a10f89

addr and connector_handler parameters are mandatory to instantiate ClientConnection class. · 56d55c34
Vincent Pelletier authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@962 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
56d55c34

Define is(Master|Storage|Client|Admin) method on Node class to avoid check with · 868be91a

Grégory Wisniewski authored Jul 23, 2009

constants from protocol module. Use them in some files, more to come.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@961 71dcc9de-d417-0410-9af5-da40c76e7ee4

868be91a

Both addr and connection_handler are mandatory constructor parameters for a ListeningConnection. · 6910e589
Vincent Pelletier authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@960 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
6910e589

Fix typo. · 67d8699b

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@959 71dcc9de-d417-0410-9af5-da40c76e7ee4

67d8699b

Don't shadow built-ins. · db0f8473

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@958 71dcc9de-d417-0410-9af5-da40c76e7ee4

db0f8473

Don't shadow filter built-in function, and using it. · 7b3d9eea
Grégory Wisniewski authored Jul 23, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@957 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
7b3d9eea

Remove unnecessary pass statements. · ec30d8f5

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@956 71dcc9de-d417-0410-9af5-da40c76e7ee4

ec30d8f5

Don't use more than one statement per line. · dcc9fbce

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@955 71dcc9de-d417-0410-9af5-da40c76e7ee4

dcc9fbce

Fix handler access. · 46950e71

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@954 71dcc9de-d417-0410-9af5-da40c76e7ee4

46950e71

Remove cyclic imports found by pylint. · 3cbc4ef7

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@953 71dcc9de-d417-0410-9af5-da40c76e7ee4

3cbc4ef7

Revert commit 950: the missing space after '*' operator was not detected by... · 203fb855

Vincent Pelletier authored Jul 23, 2009

Revert commit 950: the missing space after '*' operator was not detected by pylint because in that use it's not a binary operator, so coding style doesn't require them to be followed by a space.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@952 71dcc9de-d417-0410-9af5-da40c76e7ee4

203fb855

Add a XXX comment. · ea0a6843

Grégory Wisniewski authored Jul 23, 2009


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@951 71dcc9de-d417-0410-9af5-da40c76e7ee4

ea0a6843

22 Jul, 2009 10 commits
- Add missing space after operators. *NOT* found by pylint. · d2aed947
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@950 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  d2aed947
- Remove raise of NotImplementedError in base storage handler since sub handlers · e1292c5a
  Grégory Wisniewski authored Jul 22, 2009
```
don't always have to override those methods. 


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@949 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  e1292c5a
- Remove some unused imports found by pylint. · 9e55b249
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@948 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  9e55b249
- Remove useless decorator and old FIXME. · 99490a92
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@947 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  99490a92
- Append a space after an operator. Found by pylint. · 169cbd6b
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@946 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  169cbd6b
- This decorator is no more usefull and thus the import too. · 5275759c
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@945 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  5275759c
- Nodes use update() method of partition table to reduce duplicate code. · ec4ca515
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@944 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  ec4ca515
- Add an update() method on the common partition table. · f106a20c
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@943 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  f106a20c
- Master node don't have to process notifications from client node when · 136132cf
  Grégory Wisniewski authored Jul 22, 2009
```
shutdowning.


git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@942 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  136132cf
- Client node use update() from node manager. · 61529ea6
  Grégory Wisniewski authored Jul 22, 2009
```
git-svn-id: https://svn.erp5.org/repos/neo/branches/prototype3@941 71dcc9de-d417-0410-9af5-da40c76e7ee4
```
  61529ea6