Use /info for resolution API (#2136)

This commit is contained in:
Pascal Vizeli 2020-10-16 13:07:02 +02:00 committed by GitHub
parent 7dcb609fd5
commit 7a9aac491e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 4 additions and 5 deletions

View File

@ -199,7 +199,7 @@ class RestAPI(CoreSysAttributes):
self.webapp.add_routes( self.webapp.add_routes(
[ [
web.get("/resolution", api_resolution.base), web.get("/resolution/info", api_resolution.info),
web.post( web.post(
"/resolution/suggestion/{suggestion}", "/resolution/suggestion/{suggestion}",
api_resolution.apply_suggestion, api_resolution.apply_suggestion,

View File

@ -14,7 +14,7 @@ class APIResoulution(CoreSysAttributes):
"""Handle REST API for resoulution.""" """Handle REST API for resoulution."""
@api_process @api_process
async def base(self, request: web.Request) -> Dict[str, Any]: async def info(self, request: web.Request) -> Dict[str, Any]:
"""Return network information.""" """Return network information."""
return { return {
ATTR_UNSUPPORTED: self.sys_resolution.unsupported, ATTR_UNSUPPORTED: self.sys_resolution.unsupported,

View File

@ -44,7 +44,6 @@ NO_SECURITY_CHECK = re.compile(
OBSERVER_CHECK = re.compile( OBSERVER_CHECK = re.compile(
r"^(?:" r"^(?:"
r"|/[^/]+/info" r"|/[^/]+/info"
r"|/resolution.*"
r")$" r")$"
) )
@ -95,7 +94,7 @@ ADDONS_ROLE_ACCESS = {
r"|/network/.+" r"|/network/.+"
r"|/observer/.+" r"|/observer/.+"
r"|/os/.+" r"|/os/.+"
r"|/resolution.*" r"|/resolution/.+"
r"|/snapshots.*" r"|/snapshots.*"
r"|/supervisor/.+" r"|/supervisor/.+"
r")$" r")$"

View File

@ -24,7 +24,7 @@ async def test_api_resolution_base(coresys: CoreSys, api_client):
) )
coresys.resolution.create_issue(IssueType.FREE_SPACE, ContextType.SYSTEM) coresys.resolution.create_issue(IssueType.FREE_SPACE, ContextType.SYSTEM)
resp = await api_client.get("/resolution") resp = await api_client.get("/resolution/info")
result = await resp.json() result = await resp.json()
assert UnsupportedReason.OS in result["data"][ATTR_UNSUPPORTED] assert UnsupportedReason.OS in result["data"][ATTR_UNSUPPORTED]
assert ( assert (