feat: Added additional tests to confirm resolver and fixed uninstall script

Author: kevinwang5658

scripts/uninstall.sh

@@ -6,8 +6,19 @@ if [[ $(/usr/bin/id -u) -ne 0 ]]; then
     exit
 fi
 
-BIN_LOCATION=$(which codify)
+BIN_LOCATION=$(which codify) || { echo "Codify not found! Nothing to uninstall"; exit 0; }
 LIB_LOCATION=$(dirname "$(readlink -f $BIN_LOCATION)")
 
-sudo bash "$LIB_LOCATION/uninstall"
-rm -rf $HOME/.codify
+echo "Removing $BIN_LOCATION"
+rm -f $BIN_LOCATION
+
+echo "Removing $(realpath $LIB_LOCATION/..)"
+rm -rf "$(realpath "$LIB_LOCATION"/..)"
+
+echo "Removing \$HOME/.codify"
+rm -rf "$HOME/.codify"
+
+echo "Removing \$HOME/.local/share/codify/"
+rm -rf $HOME/.local/share/codify/
+
+echo "Done uninstalling codify. We're sorry to see you go."

test/integration/resolver.test.ts

@@ -35,6 +35,32 @@ describe('Plugin resolver integration test', () => {
     expect(pluginFolder[0]).to.eq('index.js')
   })
 
+  it ('works after .DS_store exists in the plugins folder', async() => {
+    const pluginsFolder = path.resolve(os.homedir(), '.codify', 'plugins');
+    fs.mkdirSync(pluginsFolder, { recursive: true });
+    fs.writeFileSync(path.resolve(pluginsFolder, '.DS_store'), 'ghjh');
+
+    await PluginResolver.resolveAll({ 'default': 'latest' })
+
+    expect(fs.readdirSync(pluginsFolder)).toMatchObject([
+      '.DS_store',
+      'default'
+    ])
+
+    const versions = fs.readdirSync(path.resolve(pluginsFolder, 'default'));
+    const defaultPluginFolder = path.resolve(pluginsFolder, 'default');
+    expect(versions.length).to.eq(1);
+
+    fs.rmSync(path.resolve(defaultPluginFolder, versions[0]), { recursive: true, force: true });
+    fs.writeFileSync(path.resolve(defaultPluginFolder, '.DS_store'), 'ghjh');
+
+    await PluginResolver.resolveAll({ 'default': 'latest' })
+    expect(fs.readdirSync(defaultPluginFolder)).toMatchObject([
+      '.DS_store',
+      versions[0],
+    ])
+  })
+
   afterEach(() => {
     vi.resetAllMocks();
   })