masa-finance / masa-oracle

Masa Oracle: Decentralized Data Protocol 🌐
https://developers.masa.ai/docs/masa-protocol/welcome
MIT License
24 stars 19 forks source link

fix(contracts): load config from embedded #602

Closed mudler closed 1 month ago

mudler commented 1 month ago

Description

There was still some code reading from the filesystem instead of the embedded files in the binary. Regression introduced in https://github.com/masa-finance/masa-oracle/pull/523.

Fixes: https://github.com/masa-finance/masa-oracle/issues/578

Notes for Reviewers

It also adds a simple unit test to check whether we can read from the embedded contracts. Since I was at it I also disabled the Twitter tests until properly fixed with https://github.com/masa-finance/masa-oracle/pull/573

See also: https://github.com/masa-finance/masa-oracle/pull/579

Signed commits

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 9.80%. Comparing base (532ffab) to head (504e92f). Report is 10 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #602 +/- ## ======================================== - Coverage 9.96% 9.80% -0.16% ======================================== Files 95 99 +4 Lines 5943 5892 -51 ======================================== - Hits 592 578 -14 + Misses 5276 5241 -35 + Partials 75 73 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.