Fix photosphere viewer not loading in NC33+

[Copilot]

• Fix XmpResultModel::xmlSerialize() to write a JSON string as text content instead of XML sub-elements
• Update fileAction.js _getDavXmpMeta() to JSON.parse the property value
• Fix enabled check in fileAction.js to accept both boolean true and integer 1 for usePanoramaViewer
• Update XmpResultModelTest::testXmlSerialize() to expect JSON string output
• Fix PageControllerTest: update constructor call to match new PageController(IURLGenerator, IAppManager, ContentSecurityPolicyNonceManager) signature (NC34 removed $appName/$request from controller constructor); remove stale unsafe-eval CSP assertion
• Fix PhotosphereViewerPluginTest: replace new File(...) / new Directory(...) with createMock() to avoid NC34 constructor throwing RuntimeException when view's getRelativePath() returns null; fix getId() return type from string to int (?int in NC34)

[R0Wi]

@copilot Fix failing unit tests

https://github.com/nextcloud/files_photospheres/actions/runs/25134833689/job/73670375329

[Copilot]

@copilot Fix failing unit tests

https://github.com/nextcloud/files_photospheres/actions/runs/25134833689/job/73670375329

Fixed in commit 4612e79. Two categories of failures:

1. PageControllerTest (3 errors): PageController.__construct() changed in NC34 — removed the $appName/$request parameters from the base Controller class; now takes (IURLGenerator, IAppManager, ContentSecurityPolicyNonceManager). Updated the test setup accordingly and removed the stale unsafe-eval CSP assertion.

2. PhotosphereViewerPluginTest (6 errors):

   • new File(...) / new Directory(...) throw RuntimeException: Failed to get relative path for in NC34 because Node::__construct() now validates $view->getRelativePath() returns non-null. Replaced with $this->createMock(File::class) / createMock(Directory::class) and mock the relevant methods directly.
   • Node::getId() return type changed to ?int — changed ->willReturn('42') → ->willReturn(42).

[alirezamirsepassi]

@R0Wi is there a chance this fix can be released soon?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!