This is an automated email from the ASF dual-hosted git repository.

erisu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cordova-android.git


The following commit(s) were added to refs/heads/master by this push:
     new 484c60e4 test(plugins): remove old deprecated android_studio option 
(#1815)
484c60e4 is described below

commit 484c60e48362952d26ce6b0dd90116f5811c27b7
Author: エリス <er...@users.noreply.github.com>
AuthorDate: Tue Jun 24 12:06:00 2025 +0900

    test(plugins): remove old deprecated android_studio option (#1815)
---
 spec/unit/pluginHandlers/handlers.spec.js | 62 +++++++++++++++----------------
 1 file changed, 31 insertions(+), 31 deletions(-)

diff --git a/spec/unit/pluginHandlers/handlers.spec.js 
b/spec/unit/pluginHandlers/handlers.spec.js
index 91393792..d3e0b70c 100644
--- a/spec/unit/pluginHandlers/handlers.spec.js
+++ b/spec/unit/pluginHandlers/handlers.spec.js
@@ -113,73 +113,73 @@ describe('android project handler', function () {
 
             // TODO: renumber these tests and other tests below
             it('Test#00a6 : should allow installing sources with new app 
target-dir scheme', function () {
-                android['source-file'].install(valid_source[1], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[1], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy)
                     .toHaveBeenCalledWith(dummyplugin, 
'src/android/DummyPlugin2.java', temp, 
path.join('app/src/main/src/com/phonegap/plugins/dummyplugin/DummyPlugin2.java'),
 false);
             });
 
             it('Test#006b : should allow installing jar lib file from sources 
with new app target-dir scheme', function () {
-                android['source-file'].install(valid_source[2], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[2], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy)
                     .toHaveBeenCalledWith(dummyplugin, 
'src/android/TestLib.jar', temp, path.join('app/libs/TestLib.jar'), false);
             });
 
             it('Test#006c : should allow installing aar lib file from sources 
with new app target-dir scheme', function () {
-                android['source-file'].install(valid_source[3], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[3], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy)
                     .toHaveBeenCalledWith(dummyplugin, 
'src/android/TestAar.aar', temp, path.join('app/libs/TestAar.aar'), false);
             });
 
             it('Test#006d : should allow installing xml file from sources with 
old target-dir scheme', function () {
-                android['source-file'].install(valid_source[4], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[4], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/mysettings.xml', temp,
                     path.join('app/src/main/res/xml/mysettings.xml'), false);
             });
 
             it('Test#006e : should allow installing file with other extension 
from sources with old target-dir scheme', function () {
-                android['source-file'].install(valid_source[5], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[5], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/other.extension', temp,
                     path.join('app/src/main/res/values/other.extension'), 
false);
             });
 
             it('Test#006f : should allow installing aidl file from sources 
with old target-dir scheme (GH-547)', function () {
-                android['source-file'].install(valid_source[6], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[6], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/myapi.aidl', temp,
                     path.join('app/src/main/aidl/com/mytest/myapi.aidl'), 
false);
             });
 
             it('Test#006g : should allow installing aar lib file from sources 
with old target-dir scheme (GH-547)', function () {
-                android['source-file'].install(valid_source[7], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[7], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/testaar2.aar', temp,
                     path.join('app/libs/testaar2.aar'), false);
             });
 
             it('Test#006h : should allow installing jar lib file from sources 
with old target-dir scheme (GH-547)', function () {
-                android['source-file'].install(valid_source[8], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[8], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/testjar2.jar', temp,
                     path.join('app/libs/testjar2.jar'), false);
             });
 
             it('Test#006i : should allow installing .so lib file from sources 
with old target-dir scheme (GH-547)', function () {
-                android['source-file'].install(valid_source[9], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[9], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy).toHaveBeenCalledWith(dummyplugin,
                     'src/android/jniLibs/x86/libnative.so', temp,
                     path.join('app/src/main/jniLibs/x86/libnative.so'), false);
             });
 
             it('Test#006j : should allow installing sources with target-dir 
that includes "app"', function () {
-                android['source-file'].install(valid_source[10], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[10], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy)
                     .toHaveBeenCalledWith(dummyplugin, 
'src/android/DummyPlugin2.java', temp, 
path.join('app/src/main/java/com/appco/DummyPlugin2.java'), false);
             });
 
             it('Test#006k : should allow installing sources with target-dir 
that includes "app" in its first directory', function () {
-                android['source-file'].install(valid_source[11], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[11], 
dummyPluginInfo, dummyProject);
                 expect(copyFileSpy)
                     .toHaveBeenCalledWith(dummyplugin, 
'src/android/DummyPlugin2.java', temp, 
path.join('app/src/main/java/appco/src/DummyPlugin2.java'), false);
             });
@@ -333,62 +333,62 @@ describe('android project handler', function () {
             });
 
             it('Test#019a : should remove stuff by calling common.deleteJava 
for Android Studio projects, with specific app target-dir', function () {
-                android['source-file'].install(valid_source[1], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[1], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[1], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[1], 
dummyPluginInfo, dummyProject);
                 expect(deleteJavaSpy).toHaveBeenCalledWith(temp, 
path.join('app/src/main/src/com/phonegap/plugins/dummyplugin/DummyPlugin2.java'));
             });
 
             it('Test#019b : should remove stuff by calling removeFileF for 
Android Studio projects, of jar with new app target-dir scheme', function () {
-                android['source-file'].install(valid_source[2], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[2], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[2], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[2], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/libs/TestLib.jar'), { recursive: true, force: true });
             });
 
             it('Test#019c : should remove stuff by calling removeFileF for 
Android Studio projects, of aar with new app target-dir scheme', function () {
-                android['source-file'].install(valid_source[3], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[3], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[3], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[3], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/libs/TestAar.aar'), { recursive: true, force: true });
             });
 
             it('Test#019d : should remove stuff by calling removeFileF for 
Android Studio projects, of xml with old target-dir scheme', function () {
-                android['source-file'].install(valid_source[4], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[4], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[4], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[4], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/src/main/res/xml/mysettings.xml'), { recursive: true, force: true });
             });
 
             it('Test#019e : should remove stuff by calling removeFileF for 
Android Studio projects, of file with other extension with old target-dir 
scheme', function () {
-                android['source-file'].install(valid_source[5], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[5], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[5], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[5], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/src/main/res/values/other.extension'), { recursive: true, force: true });
             });
 
             it('Test#019f : should remove stuff by calling removeFileF for 
Android Studio projects, of aidl with old target-dir scheme (GH-547)', function 
() {
-                android['source-file'].install(valid_source[6], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[6], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[6], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[6], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/src/main/aidl/com/mytest/myapi.aidl'), { recursive: true, force: true });
             });
 
             it('Test#019g : should remove stuff by calling removeFileF for 
Android Studio projects, of aar with old target-dir scheme (GH-547)', function 
() {
-                android['source-file'].install(valid_source[7], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[7], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[7], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[7], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/libs/testaar2.aar'), { recursive: true, force: true });
             });
 
             it('Test#019h : should remove stuff by calling removeFileF for 
Android Studio projects, of jar with old target-dir scheme (GH-547)', function 
() {
-                android['source-file'].install(valid_source[8], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[8], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[8], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[8], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/libs/testjar2.jar'), { recursive: true, force: true });
             });
 
             it('Test#019i : should remove stuff by calling removeFileF for 
Android Studio projects, of .so lib file with old target-dir scheme (GH-547)', 
function () {
-                android['source-file'].install(valid_source[9], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[9], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[9], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[9], 
dummyPluginInfo, dummyProject);
                 
expect(rmSyncSpy).toHaveBeenCalledWith(path.join(dummyProject.projectDir, 
'app/src/main/jniLibs/x86/libnative.so'), { recursive: true, force: true });
             });
 
             it('Test#019j : should remove stuff by calling common.deleteJava 
for Android Studio projects, with target-dir that includes "app"', function () {
-                android['source-file'].install(valid_source[10], 
dummyPluginInfo, dummyProject, { android_studio: true });
-                android['source-file'].uninstall(valid_source[10], 
dummyPluginInfo, dummyProject, { android_studio: true });
+                android['source-file'].install(valid_source[10], 
dummyPluginInfo, dummyProject);
+                android['source-file'].uninstall(valid_source[10], 
dummyPluginInfo, dummyProject);
                 
expect(deleteJavaSpy).toHaveBeenCalledWith(dummyProject.projectDir, 
path.join('app/src/main/java/com/appco/DummyPlugin2.java'));
             });
         });


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cordova.apache.org
For additional commands, e-mail: commits-h...@cordova.apache.org

Reply via email to