[PR #4125] [CLOSED] Improve test coverage #3822

Closed
opened 2026-02-26 08:32:05 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4125
Author: @snordmann
Created: 10/31/2024
Status: Closed

Base: developHead: more-tests


📝 Commits (2)

  • c8c95ba Add more tests for proxy_hosts
  • fac7bcc Add initial frontend test for login

📊 Changes

2 files changed (+126 additions, -2 deletions)

View changed files

📝 test/cypress/e2e/api/ProxyHosts.cy.js (+112 -2)
test/cypress/e2e/frontend/Login.cy.js (+14 -0)

📄 Description

In this PR my goal is to test more endpoints and maybe even the frontend. This should make verification of future changes easier.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4125 **Author:** [@snordmann](https://github.com/snordmann) **Created:** 10/31/2024 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `more-tests` --- ### 📝 Commits (2) - [`c8c95ba`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/c8c95baed1ccad48c3a41f199e082fcec15ce3dc) Add more tests for proxy_hosts - [`fac7bcc`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/fac7bcca0282f04d3d0b4a68d3f2d6324c5ff47a) Add initial frontend test for login ### 📊 Changes **2 files changed** (+126 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `test/cypress/e2e/api/ProxyHosts.cy.js` (+112 -2) ➕ `test/cypress/e2e/frontend/Login.cy.js` (+14 -0) </details> ### 📄 Description In this PR my goal is to test more endpoints and maybe even the frontend. This should make verification of future changes easier. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 08:32:05 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nginx-proxy-manager-NginxProxyManager#3822
No description provided.