Code review comment for lp:~al-maisan/landscape-client/obsolete-results-v

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

The branch looks great! Thanks for addressing all the comments. +1

[14]

+ test_case.exchange_store = ExchangeStore(
+ os.path.join(test_case.config.data_path, "exchange.database"))

You can now use BrokerConfiguration.exchange_store_path:

+ test_case.exchange_store = ExchangeStore(
+ test_case.config.exchange_store_path)

review: Approve

« Back to merge proposal