[useAxios] Test case missing url doesn't pass.

[yysuen]

Describe the bug

In a word

Here's the original output of Vitest.

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Tests 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  packages/integrations/useAxios/index.test.ts > useAxios > missing url
AssertionError: expected undefined not to be undefined
 ❯ packages/integrations/useAxios/index.test.ts:281:25
    279|     const { execute } = useAxios(und…
    280|     const { error } = await execute()
    281|     expect(error.value).toBeDefined()
       |                         ^
    282|   })
    283| })

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed | 125 passed (126)
      Tests  1 failed | 813 passed (814)
   Start at  19:11:09
   Duration  99.82s


 FAIL  Tests failed. Watching for file changes...

About the reason

After logging, I found that error.value here is undefined.

The reason is that in execute function, _url is assigned a default '' (empty string) when it's undefined. Therefore, axios doesn't walk into the catch branch.

My solution

I'll open a PR later and mention it below.

Reproduction

Clone https://github.com/syy11cn/vueuse and run pnpm run test.

System Info

System:
    OS: macOS 13.0.1
    CPU: (8) arm64 Apple M2
    Memory: 59.03 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.17.0 - ~/.nvm/versions/node/v16.17.0/bin/node
    npm: 8.15.0 - ~/.nvm/versions/node/v16.17.0/bin/npm
  Browsers:
    Chrome: 107.0.5304.110
    Safari: 16.1

Used Package Manager

pnpm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a VueUse issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion.
• The provided reproduction is a minimal reproducible example of the bug.