Remove surface_integral from prolong2interfaces

[benegee]

Xref: trixi-framework/Trixi.jl#2434

This was not caught in Trixi.jl's downstream tests as we currently only run one Euler elixir.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.68%. Comparing base (8e9054c) to head (080581a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #99   +/-   ##
=======================================
  Coverage   90.68%   90.68%           
=======================================
  Files          22       22           
  Lines        2189     2189           
=======================================
  Hits         1985     1985           
  Misses        204      204           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[amrueda]

Thanks for fixing this, @benegee!

This was not caught in Trixi.jl's downstream tests as we currently only run one Euler elixir.

I don't fully understand why this wasn't caught. Shouldn't that single elixir fail to find the function prolong2interfaces!?

[benegee]

Well, I was not very precise.

prolong2interfaces originally lives in Trxi.jl, unless you override it, as done here for the solvers on manifolds. We, however, do not test these solvers in the downstream tests. I just used the warm bubble test there, based on CompressibleEulerEquations, for which we did not override rhs or pronlong2interface.

We can, and probably should, add further meaningful tests to the downstream pipeline.