stfc / rucio

Rucio - Scientific Data Management
http://rucio.cern.ch
Apache License 2.0
0 stars 0 forks source link

internal bug low-priority: Prevent overwriting default dict #43 #46

Closed patrick-austin closed 4 years ago

patrick-austin commented 4 years ago

internal bug low-priority: Prevent overwriting default dict #43

See issue #43 for a detailed description of the change.

patrick-austin commented 4 years ago

Following discussion with Martin it was decided to leave existing examples of filters={} alone, and that in general issues arising in tests should be dealt with by passing an empty dictionary on function calls. We did agree (I think?) to at least leave in the logic for creating a new dictionary in places that have been affected by multi_vo changes (i.e. the three places addressed by this PR) as these could lead to bugs.