Skip to content
This repository was archived by the owner on Jul 2, 2026. It is now read-only.

Skip e2e when missing API key - #2

Merged
arran4 merged 1 commit into
mainfrom
codex/skip-e2e-tests-if-gemini_api_key-is-missing
Jun 26, 2025
Merged

Skip e2e when missing API key#2
arran4 merged 1 commit into
mainfrom
codex/skip-e2e-tests-if-gemini_api_key-is-missing

Conversation

@arran4

@arran4 arran4 commented Jun 26, 2025

Copy link
Copy Markdown
Owner

Summary

  • skip e2e tests if the GEMINI_API_KEY secret isn't set
  • document that the workflow skips e2e tests when the API key is missing

Testing

  • npm run lint (fails: Cannot find package '@eslint/js')

https://chatgpt.com/codex/tasks/task_e_685c8ee017d0832f816d496151f3acd5

@arran4
arran4 merged commit 6570ee6 into main Jun 26, 2025
1 check passed
@arran4
arran4 deleted the codex/skip-e2e-tests-if-gemini_api_key-is-missing branch June 26, 2025 00:10
@github-actions

github-actions Bot commented Jun 26, 2025

Copy link
Copy Markdown

Code Coverage Summary

Package Lines Statements Functions Branches
CLI 68.06% 68.06% 72% 75.08%
Core 70.06% 70.06% 74.55% 80.97%
CLI Package - Full Text Report 
-------------------|---------|----------|---------|---------|-------------------
File               | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-------------------|---------|----------|---------|---------|-------------------
All files          |   68.06 |    75.08 |      72 |   68.06 |                   
 src               |   42.01 |    53.12 |    37.5 |   42.01 |                   
  gemini.tsx       |   16.74 |    14.28 |   16.66 |   16.74 | ...67-216,221-292 
  ...ractiveCli.ts |   89.74 |       64 |     100 |   89.74 | ...22,125,148-149 
 src/config        |   80.87 |    67.56 |   81.48 |   80.87 |                   
  auth.ts          |   15.38 |      100 |       0 |   15.38 | 11-39             
  config.ts        |   91.48 |    68.42 |   66.66 |   91.48 | ...89-290,300-301 
  extension.ts     |    73.8 |    70.58 |     100 |    73.8 | ...01-105,114-115 
  sandboxConfig.ts |   51.35 |    16.66 |   66.66 |   51.35 | ...43,53-69,74-91 
  settings.ts      |   92.44 |    79.54 |     100 |   92.44 | ...25-226,262-263 
 src/generated     |     100 |      100 |     100 |     100 |                   
  git-commit.ts    |     100 |      100 |     100 |     100 |                   
 src/ui            |   59.81 |    55.26 |   71.42 |   59.81 |                   
  App.tsx          |   57.37 |     45.9 |   71.42 |   57.37 | ...80-791,797-826 
  colors.ts        |   86.04 |      100 |   76.92 |   86.04 | 12-13,18-19,42-43 
  constants.ts     |       0 |        0 |       0 |       0 | 1-15              
  types.ts         |     100 |      100 |     100 |     100 |                   
 src/ui/components |   60.32 |    57.06 |   57.14 |   60.32 |                   
  AboutBox.tsx     |     100 |      100 |     100 |     100 |                   
  AsciiArt.ts      |     100 |      100 |     100 |     100 |                   
  AuthDialog.tsx   |   90.12 |      100 |      50 |   90.12 | 50-57             
  ...nProgress.tsx |   15.78 |      100 |       0 |   15.78 | 17-57             
  ...Indicator.tsx |   15.15 |      100 |       0 |   15.15 | 17-47             
  ...lePatcher.tsx |   73.17 |      100 |   66.66 |   73.17 | 34-46             
  ...ryDisplay.tsx |   21.05 |      100 |       0 |   21.05 | 17-35             
  ...ryDisplay.tsx |   97.82 |    95.83 |     100 |   97.82 | 59                
  ...esDisplay.tsx |   10.52 |      100 |       0 |   10.52 | 24-82             
  ...ngsDialog.tsx |    6.76 |      100 |       0 |    6.76 | 26-168            
  Footer.tsx       |   71.42 |    11.11 |     100 |   71.42 | ...,92-99,102-105 
  ...ngSpinner.tsx |      80 |    33.33 |     100 |      80 | 29,31-32          
  Header.tsx       |   96.87 |       60 |     100 |   96.87 | 27                
  Help.tsx         |    4.81 |      100 |       0 |    4.81 | 17-128            
  ...emDisplay.tsx |   68.65 |       50 |     100 |   68.65 | ...53-58,81-88,91 
  InputPrompt.tsx  |   57.22 |    39.74 |     100 |   57.22 | ...11-415,426-435 
  ...Indicator.tsx |     100 |      100 |     100 |     100 |                   
  ...geDisplay.tsx |   25.92 |      100 |       0 |   25.92 | 14-36             
  ...ryDisplay.tsx |      94 |       50 |     100 |      94 | 50-52             
  ...Indicator.tsx |   44.44 |      100 |       0 |   44.44 | 12-17             
  ...MoreLines.tsx |      60 |       25 |     100 |      60 | 24-27,33-40       
  Stats.tsx        |     100 |      100 |     100 |     100 |                   
  StatsDisplay.tsx |     100 |      100 |     100 |     100 |                   
  ...nsDisplay.tsx |    8.47 |      100 |       0 |    8.47 | 26-93             
  ThemeDialog.tsx  |    82.7 |       20 |      25 |    82.7 | ...,95-99,197-208 
  Tips.tsx         |     100 |      100 |     100 |     100 |                   
  ...ification.tsx |   36.36 |      100 |       0 |   36.36 | 15-22             
 ...nents/messages |   62.34 |     80.9 |   57.89 |   62.34 |                   
  ...onMessage.tsx |   18.51 |      100 |       0 |   18.51 | 22-49             
  DiffRenderer.tsx |   96.31 |    82.66 |     100 |   96.31 | ...01-202,206,271 
  ErrorMessage.tsx |   22.22 |      100 |       0 |   22.22 | 16-31             
  ...niMessage.tsx |   18.51 |      100 |       0 |   18.51 | 20-43             
  ...geContent.tsx |   19.04 |      100 |       0 |   19.04 | 25-43             
  InfoMessage.tsx  |     100 |      100 |     100 |     100 |                   
  ...onMessage.tsx |   38.91 |     62.5 |   33.33 |   38.91 | ...32-164,200-225 
  ...upMessage.tsx |     9.3 |      100 |       0 |     9.3 | 26-123            
  ToolMessage.tsx  |   87.76 |       80 |     100 |   87.76 | ...,91-95,169-171 
  UserMessage.tsx  |     100 |      100 |     100 |     100 |                   
  ...llMessage.tsx |   36.36 |      100 |       0 |   36.36 | 17-25             
 ...ponents/shared |   74.06 |    72.98 |    82.6 |   74.06 |                   
  MaxSizedBox.tsx  |   78.49 |     79.8 |   88.88 |   78.49 | ...21-423,522-523 
  ...tonSelect.tsx |   88.05 |    92.85 |      60 |   88.05 | ...,68-71,108-109 
  text-buffer.ts   |   71.84 |    68.69 |   88.88 |   71.84 | ...1393,1396-1397 
 src/ui/contexts   |   95.58 |    66.66 |     100 |   95.58 |                   
  ...owContext.tsx |   91.07 |    81.81 |     100 |   91.07 | 46-47,59-61       
  ...onContext.tsx |     100 |    58.82 |     100 |     100 | 65-71             
  ...ngContext.tsx |   71.42 |       50 |     100 |   71.42 | 17-20             
 src/ui/editors    |   93.33 |    85.71 |   66.66 |   93.33 |                   
  ...ngsManager.ts |   93.33 |    85.71 |   66.66 |   93.33 | 49,63-64          
 src/ui/hooks      |   71.33 |    82.41 |   70.31 |   71.33 |                   
  ...dProcessor.ts |   81.87 |    82.75 |     100 |   81.87 | ...94-397,408-424 
  ...dProcessor.ts |      82 |    74.35 |      80 |      82 | ...15-324,328-329 
  ...dProcessor.ts |   65.68 |     80.5 |      55 |   65.68 | ...1010,1024-1025 
  ...uthCommand.ts |     5.4 |      100 |       0 |     5.4 | 16-19,22-97       
  ...tIndicator.ts |     100 |      100 |     100 |     100 |                   
  useCompletion.ts |   50.99 |    81.13 |      60 |   50.99 | ...41-346,411-414 
  ...leMessages.ts |   96.92 |    88.88 |     100 |   96.92 | 25-26             
  ...orSettings.ts |     100 |      100 |     100 |     100 |                   
  ...miniStream.ts |   67.86 |    76.51 |     100 |   67.86 | ...65,667,718-808 
  ...BranchName.ts |   91.66 |    84.61 |     100 |   91.66 | 57-63             
  ...oryManager.ts |   98.41 |    93.33 |     100 |   98.41 | 43                
  ...putHistory.ts |    92.5 |    85.71 |     100 |    92.5 | 62-63,71,93-95    
  ...gIndicator.ts |     100 |      100 |     100 |     100 |                   
  useLogger.ts     |      25 |      100 |       0 |      25 | 14-32             
  ...raseCycler.ts |    95.5 |       75 |     100 |    95.5 | ...66-167,185-187 
  ...lScheduler.ts |   79.01 |    94.87 |     100 |   79.01 | ...00-203,293-303 
  ...oryCommand.ts |       0 |        0 |       0 |       0 | 1-7               
  ...ellHistory.ts |   91.95 |    79.16 |   83.33 |   91.95 | 28-30,41-42,87-88 
  ...oryCommand.ts |    4.91 |      100 |       0 |    4.91 | 12-75             
  ...tateAndRef.ts |   95.45 |    66.66 |     100 |   95.45 | 25                
  ...rminalSize.ts |   77.27 |      100 |      50 |   77.27 | 19-23             
  ...emeCommand.ts |   64.28 |     87.5 |     100 |   64.28 | ...,92-93,100-106 
  useTimer.ts      |   88.09 |    85.71 |     100 |   88.09 | 44-45,51-53       
 src/ui/themes     |   99.34 |    89.74 |     100 |   99.34 |                   
  ansi-light.ts    |     100 |      100 |     100 |     100 |                   
  ansi.ts          |     100 |      100 |     100 |     100 |                   
  atom-one-dark.ts |     100 |      100 |     100 |     100 |                   
  ayu-light.ts     |     100 |      100 |     100 |     100 |                   
  ayu.ts           |     100 |      100 |     100 |     100 |                   
  default-light.ts |     100 |      100 |     100 |     100 |                   
  default.ts       |     100 |      100 |     100 |     100 |                   
  dracula.ts       |     100 |      100 |     100 |     100 |                   
  github-dark.ts   |     100 |      100 |     100 |     100 |                   
  github-light.ts  |     100 |      100 |     100 |     100 |                   
  googlecode.ts    |     100 |      100 |     100 |     100 |                   
  no-color.ts      |     100 |      100 |     100 |     100 |                   
  theme-manager.ts |   89.53 |    84.21 |     100 |   89.53 | 64,96-101,106-107 
  theme.ts         |   98.44 |       95 |     100 |   98.44 | 304-307           
  xcode.ts         |     100 |      100 |     100 |     100 |                   
 src/ui/utils      |   44.01 |    90.47 |   83.33 |   44.01 |                   
  ...Colorizer.tsx |    80.5 |    86.95 |     100 |    80.5 | 88-89,159-183     
  ...wnDisplay.tsx |    6.31 |      100 |       0 |    6.31 | ...48-406,418-442 
  commandUtils.ts  |     100 |      100 |     100 |     100 |                   
  errorParsing.ts  |     100 |      100 |     100 |     100 |                   
  formatters.ts    |   90.47 |    95.83 |     100 |   90.47 | 57-60             
  ...nUtilities.ts |   69.84 |    85.71 |     100 |   69.84 | 75-91,100-101     
  textUtils.ts     |   93.93 |    92.85 |     100 |   93.93 | 14-15             
  updateCheck.ts   |   30.76 |       25 |     100 |   30.76 | 14-35             
 src/utils         |    5.29 |    66.66 |      50 |    5.29 |                   
  cleanup.ts       |   91.66 |       50 |     100 |   91.66 | 18                
  package.ts       |   88.88 |    85.71 |     100 |   88.88 | 33-34             
  readStdin.ts     |    3.44 |      100 |       0 |    3.44 | 7-39              
  sandbox.ts       |       0 |        0 |       0 |       0 | 1-871             
  ...upWarnings.ts |   23.07 |      100 |       0 |   23.07 | 14-40             
  version.ts       |     100 |       50 |     100 |     100 | 11                
-------------------|---------|----------|---------|---------|-------------------
Core Package - Full Text Report 
-------------------|---------|----------|---------|---------|-------------------
File               | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-------------------|---------|----------|---------|---------|-------------------
All files          |   70.06 |    80.97 |   74.55 |   70.06 |                   
 src               |     100 |      100 |     100 |     100 |                   
  index.ts         |     100 |      100 |     100 |     100 |                   
 src/__mocks__/fs  |     100 |      100 |     100 |     100 |                   
  promises.ts      |     100 |      100 |     100 |     100 |                   
 src/code_assist   |   64.94 |    78.94 |   73.33 |   64.94 |                   
  codeAssist.ts    |   31.25 |      100 |       0 |   31.25 | 12-23             
  converter.ts     |    88.8 |    96.15 |   81.81 |    88.8 | 171-175,194-204   
  oauth2.ts        |   80.13 |       60 |   85.71 |   80.13 | ...67-178,196-202 
  server.ts        |   54.47 |       80 |   77.77 |   54.47 | 111-127,130-171   
  setup.ts         |    5.55 |      100 |       0 |    5.55 | 16-57             
  types.ts         |       0 |        0 |       0 |       0 | 1-160             
 src/config        |   66.37 |    88.67 |   49.01 |   66.37 |                   
  config.ts        |   65.97 |    88.67 |   49.01 |   65.97 | ...50-455,459-499 
  models.ts        |     100 |      100 |     100 |     100 |                   
 src/core          |   65.54 |    76.01 |   79.31 |   65.54 |                   
  client.ts        |   55.92 |    64.58 |   81.81 |   55.92 | ...39-499,508-536 
  ...tGenerator.ts |   40.25 |     62.5 |      50 |   40.25 | ...01,118,129-132 
  ...lScheduler.ts |   57.32 |    66.07 |   78.57 |   57.32 | ...31-561,580-638 
  geminiChat.ts    |   66.49 |    77.92 |   69.23 |   66.49 | ...45,560,564-572 
  geminiRequest.ts |     100 |      100 |     100 |     100 |                   
  logger.ts        |   84.61 |    82.66 |     100 |   84.61 | ...58-259,276-280 
  modelCheck.ts    |    4.65 |      100 |       0 |    4.65 | 20-68             
  ...olExecutor.ts |     100 |    66.66 |     100 |     100 | 63,93             
  prompts.ts       |   83.51 |    68.18 |     100 |   83.51 | 28-36,38,260-265  
  tokenLimits.ts   |      15 |      100 |       0 |      15 | 15-31             
  turn.ts          |   83.21 |    83.87 |     100 |   83.21 | ...33-236,249-250 
 src/services      |   83.66 |    86.11 |      75 |   83.66 |                   
  ...eryService.ts |   91.93 |       75 |     100 |   91.93 | 31,39,61-62,84    
  gitService.ts    |   78.02 |      100 |      60 |   78.02 | ...10-114,117-121 
 src/telemetry     |   78.33 |    84.53 |   81.81 |   78.33 |                   
  constants.ts     |     100 |      100 |     100 |     100 |                   
  index.ts         |     100 |      100 |     100 |     100 |                   
  loggers.ts       |   84.34 |       85 |    87.5 |   84.34 | 153-186           
  metrics.ts       |   60.36 |    95.65 |    62.5 |   60.36 | ...36-158,161-184 
  sdk.ts           |   82.85 |    28.57 |     100 |   82.85 | ...18,126-127,133 
  types.ts         |   82.32 |    97.43 |   84.61 |   82.32 | 83-92,149-172     
 ...learcut-logger |   82.93 |    81.25 |   72.22 |   82.93 |                   
  ...cut-logger.ts |   82.97 |    80.64 |   76.47 |   82.97 | ...81-391,394-396 
  ...tadata-key.ts |    82.6 |      100 |       0 |    82.6 | 143-153           
 src/tools         |   61.75 |    79.95 |   70.96 |   61.75 |                   
  diffOptions.ts   |     100 |      100 |     100 |     100 |                   
  edit.ts          |   77.74 |    81.15 |   84.61 |   77.74 | ...40-441,445-479 
  glob.ts          |   87.03 |       80 |   85.71 |   87.03 | ...89-290,298-305 
  grep.ts          |   56.59 |       75 |   72.72 |   56.59 | ...28-533,538-542 
  ls.ts            |    8.55 |      100 |    12.5 |    8.55 | ...89-194,202-312 
  mcp-client.ts    |   73.94 |    74.07 |   36.36 |   73.94 | ...35-336,355-356 
  mcp-tool.ts      |   91.75 |    85.71 |     100 |   91.75 | ...36,142,147-148 
  memoryTool.ts    |   97.43 |    84.84 |     100 |   97.43 | 93,95,97-98       
  ...iable-tool.ts |     100 |    84.61 |     100 |     100 | 90,97             
  read-file.ts     |   98.43 |    96.15 |     100 |   98.43 | 121-122           
  ...many-files.ts |   79.78 |    75.86 |   83.33 |   79.78 | ...67-468,475-476 
  shell.ts         |    4.04 |       50 |      50 |    4.04 | 33-365            
  tool-registry.ts |   51.23 |       80 |      80 |   51.23 | ...69-172,201-206 
  tools.ts         |   76.31 |      100 |      40 |   76.31 | ...44-145,154-159 
  web-fetch.ts     |   31.29 |    72.22 |   66.66 |   31.29 | ...90-191,219-359 
  web-search.ts    |   10.92 |      100 |      20 |   10.92 | ...01-102,105-194 
  write-file.ts    |   81.39 |       82 |   81.81 |   81.39 | ...05-310,371-400 
 src/utils         |   82.28 |    83.58 |   87.09 |   82.28 |                   
  LruCache.ts      |   70.96 |     62.5 |     100 |   70.96 | 20-22,28,30-34    
  bfsFileSearch.ts |   92.45 |    86.66 |     100 |   92.45 | 53-54,67-68       
  editCorrector.ts |   81.92 |     70.9 |     100 |   81.92 | ...42-554,588,602 
  editor.ts        |    97.1 |    93.18 |     100 |    97.1 | 118,178,181-182   
  ...rReporting.ts |   83.52 |    84.61 |     100 |   83.52 | 81-85,106-114     
  errors.ts        |   41.46 |       60 |      75 |   41.46 | 17-21,37-52,56-62 
  fetch.ts         |   34.04 |      100 |       0 |   34.04 | 22-27,31-57       
  fileUtils.ts     |   96.09 |    91.37 |     100 |   96.09 | 47,271-277        
  ...eUtilities.ts |   96.03 |       96 |     100 |   96.03 | 28-29,57-58       
  ...rStructure.ts |      95 |    93.42 |     100 |      95 | ...66-167,344-346 
  ...noreParser.ts |   96.36 |     90.9 |     100 |   96.36 | 70-71             
  gitUtils.ts      |   46.34 |    66.66 |      50 |   46.34 | 24-25,40-41,50-73 
  ...yDiscovery.ts |   80.24 |    75.43 |      75 |   80.24 | ...03-304,307-308 
  ...Inspectors.ts |     100 |      100 |     100 |     100 |                   
  ...kerChecker.ts |   83.14 |    82.35 |     100 |   83.14 | ...,94-99,107-113 
  paths.ts         |   62.79 |    73.33 |   57.14 |   62.79 | ...23-133,139-140 
  retry.ts         |   68.85 |    73.91 |     100 |   68.85 | ...89-209,254-269 
  ...aValidator.ts |     100 |      100 |     100 |     100 |                   
  session.ts       |     100 |      100 |     100 |     100 |                   
  testUtils.ts     |   84.44 |    72.72 |   83.33 |   84.44 | 27-28,34-35,70-72 
  user_id.ts       |   60.97 |    33.33 |      75 |   60.97 | ...33,46-48,52-57 
-------------------|---------|----------|---------|---------|-------------------

For detailed HTML reports, please see the 'coverage-reports-20.x' artifact from the main CI run.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

codex

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@arran4