chore(quality): type-check and lint gates in pre-deploy, clean lint/tsc
CI Pipeline with Test Management / 🧪 Run Tests & Generate Reports (push) Waiting to run
CI Pipeline with Test Management / 🐳 Docker Integration Tests (push) Blocked by required conditions
CI Pipeline with Test Management / 🏗️ Build Docker Image (push) Blocked by required conditions
CI Pipeline with Test Management / 📊 Generate Test Summary (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / 🧪 Run Tests & Generate Reports (push) Waiting to run
Test Reporting & Gherkin Analysis / 📊 Analyze Test Coverage (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / 🔄 Sync with Linear (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / ⚡ Performance Monitoring (push) Blocked by required conditions
CI Pipeline with Test Management / 🧪 Run Tests & Generate Reports (push) Waiting to run
CI Pipeline with Test Management / 🐳 Docker Integration Tests (push) Blocked by required conditions
CI Pipeline with Test Management / 🏗️ Build Docker Image (push) Blocked by required conditions
CI Pipeline with Test Management / 📊 Generate Test Summary (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / 🧪 Run Tests & Generate Reports (push) Waiting to run
Test Reporting & Gherkin Analysis / 📊 Analyze Test Coverage (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / 🔄 Sync with Linear (push) Blocked by required conditions
Test Reporting & Gherkin Analysis / ⚡ Performance Monitoring (push) Blocked by required conditions
- pre-deploy suite now runs tsc --noEmit and eslint; any failure aborts the release - fix all TypeScript errors (FooterConfig.copyright removed after the CMS move; test guards/casts) - fix all ESLint errors: <a>→<Link> and <img>→<Image> in Header/Footer, remove unused imports and explicit any in src - eslint config relaxes no-explicit-any/no-require-imports/no-unused-vars for test and CommonJS config files (legitimate usage) Closes MITHOME-81
This commit is contained in:
@@ -23,7 +23,8 @@ describe('Logger', () => {
|
||||
|
||||
it('should create logger with correct configuration', () => {
|
||||
// Set NODE_ENV to development to get debug level
|
||||
process.env.NODE_ENV = 'development'
|
||||
// NODE_ENV is typed read-only in @types/node — assign via a wider type.
|
||||
;(process.env as unknown as Record<string, string>).NODE_ENV = 'development'
|
||||
|
||||
// Import after clearing modules
|
||||
require('./logger')
|
||||
@@ -54,7 +55,7 @@ describe('Logger', () => {
|
||||
|
||||
// Clear the mock calls to avoid interference from logger creation
|
||||
const mockLogger = mockedWinston.createLogger()
|
||||
mockLogger.child.mockClear()
|
||||
;(mockLogger.child as jest.Mock).mockClear()
|
||||
|
||||
const componentLogger = createComponentLogger('test-component')
|
||||
expect(logger.child).toHaveBeenCalledWith({ component: 'test-component' })
|
||||
|
||||
@@ -92,7 +92,7 @@ export const generateRequestId = (): string =>
|
||||
export const withTiming = async <T>(
|
||||
operation: string,
|
||||
fn: () => Promise<T>,
|
||||
metadata: any = {}
|
||||
metadata: Record<string, unknown> = {}
|
||||
): Promise<T> => {
|
||||
const startTime = Date.now()
|
||||
logger.debug(`Started ${operation}`, metadata)
|
||||
|
||||
Reference in New Issue
Block a user