chore: reorganize mocks
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 38/100
Research direction
Start by comparing the mock subdirectories, especially the default 2024.08.0 directory, with their server settings and versioned peers. Read the discussion linked in the issue before choosing between renaming the default directory or removing the default mock path; done means the layout and README or test references consistently reflect the decision.
Written by the indexing model from the issue text.
Description
Most of the mocks are organized into subdirectories by Connect version. The default mock directory is named 2024.08.0 but it actually does not contain a version number in its server settings. The other mock directories do respond like the named Connect version.
Some options:
- At the very least, the default mock directory should be renamed something like
unversioned(with a note in a README to describe what that means). This might be fine for tests that use code that doesn't explicitly check the Connect version. - Another option would be to remove the default mock path and always be explicit (described by @nealrichardson here). Even code that doesn't explicitly check a version of Connect is written against some version, so it might just be good practice to put any mock in a directory for the version of Connect it was captured from.
- Dominant language
- R
- Stars
- 54
- Forks
- 27
- Avg merge
- 1d 3m
- Merged PRs (30d)
- 1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from posit-dev/connectapi
-
Difficulty 1/5 Under an hour Newbie friendliness 62/100
posit-dev/connectapi#249 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 65/100
posit-dev/connectapi#527 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
posit-dev/connectapi#526 · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
posit-dev/connectapi#506 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
posit-dev/connectapi#486 ·
All issues in posit-dev/connectapi
Similar issues
-
documentation pkg infrastructure
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
epiverse-trace/epiparameter#511 ·
-
function:write_dwc
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Urgent request: Due to vulnerabilities move to API version 12.6.1 (12.6.2 eventually) or 13.1.1 Open
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
jbkunst/highcharter#849 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
r-lib/pkgdepends#485 · 3 comments ·
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100