fix(namespace-error): Remove not needed android namespace

fixes #1

Author: hypery2k

.gitignore

@@ -1,2 +1,3 @@
 node_modules
 .DS_Store
+reports

.travis.yml

@@ -0,0 +1,18 @@
+sudo: required
+dist: trusty
+language: node_js
+node_js:
+  - "4"
+  - "5"
+  - "6"
+  - "7"
+  - "stable"
+env:
+  - NPM_VERSION=4
+  - NPM_VERSION=3
+before_install:
+  - npm install -g npm@$NPM_VERSION
+before_script:
+  - npm install
+script:
+  - npm test

index.js

@@ -1,18 +1,23 @@
 'use strict';
 
+const path = require('path');
+const fs = require('fs');
 const xmlpoke = require('xmlpoke');
 const packageJSON = require(process.cwd() + '/package.json');
 
-console.log(process.cwd())
-
 function configpluginxml(inFile) {
   const defaultFile = 'plugin.xml';
   const packageFile = 'package.json';
   const file = inFile || defaultFile;
 
+  console.log(file);
+
   function poke(cb) {
+    let xmlContent = fs.readFileSync(path.resolve(file), 'utf8');
     // https://github.com/mikeobrien/node-xmlpoke/issues/3
-    xmlpoke(file, xml => cb(xml.addNamespace('p', 'http://apache.org/cordova/ns/plugins/1.0')));
+    let updatedContent = xmlpoke(xmlContent, xml => cb(xml.addNamespace('p', 'http://apache.org/cordova/ns/plugins/1.0')));
+    updatedContent = updatedContent.replace(' xmlns:android=""', '');
+    fs.writeFileSync(path.resolve(file), updatedContent, 'utf8');
   }
 
   function set(xpath, val) {

package.json

@@ -8,7 +8,7 @@
     "clean": "rimraf node_modules",
     "postclean": "npm i",
     "build": "exit 0",
-    "test": "exit 0",
+    "test": "jasmine-node test --junitreport",
     "changelog": "conventional-changelog -p angular -i CHANGELOG.md -s -r 0",
     "changelog:add": "git add CHANGELOG.md && git commit -m 'chore(changelog): Updated CHANGELOG.md'",
     "release:pre": "npm run clean && npm run build",
@@ -61,6 +61,11 @@
   },
   "devDependencies": {
     "conventional-changelog-cli": "1.3.1",
+    "istanbul": "0.4.5",
+    "istanbul-coveralls": "1.0.3",
+    "jasmine-node": "1.14.5",
+    "jscoverage": "0.6.0",
+    "nodeunit": "0.11.0",
     "rimraf": "2.6.1"
   }
 }

test/index.spec.js

@@ -0,0 +1,15 @@
+var configPluginXml = require('../index'),
+    path = require('path'),
+    fs = require('fs');
+
+describe('api', function () {
+
+    it('should replace', function (done) {
+        configPluginXml(path.resolve(__dirname, 'test.xml')).setVersion('0.1.0');
+        setTimeout(function () {
+            var testXml = fs.readFileSync(path.resolve(__dirname, 'test.xml'), 'utf8');
+            expect(testXml).not.toContain('<activity xmlns:android=""');
+            done();
+        }, 1000);
+    });
+});

test/test.xml

@@ -0,0 +1,84 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+
+<plugin xmlns="http://apache.org/cordova/ns/plugins/1.0" id="cordova-plugin-test" version="0.1.0">
+
+    <name>test</name>
+    <description>Scans Barcodes</description>
+    <license>MIT</license>
+
+    <engines>
+        <engine name="cordova" version=">=3.0.0"/>
+    </engines>
+
+
+    <!-- ios -->
+    <platform name="ios">
+        <!-- Cordova >= 2.8 -->
+        <config-file target="config.xml" parent="/*">
+            <feature name="BarcodeScanner">
+                <param name="ios-package" value="CDVBarcodeScanner"/>
+            </feature>
+        </config-file>
+
+        <resource-file src="src/ios/scannerOverlay.xib"/>
+
+        <header-file src="src/ios/zxing-all-in-one.h"/>
+
+        <source-file src="src/ios/CDVBarcodeScanner.mm" compiler-flags="-fno-objc-arc"/>
+        <source-file src="src/ios/zxing-all-in-one.cpp"/>
+
+        <framework src="libiconv.dylib"/>
+        <framework src="AVFoundation.framework"/>
+        <framework src="AssetsLibrary.framework"/>
+        <framework src="CoreVideo.framework"/>
+        <framework src="QuartzCore.framework"/>
+    </platform>
+
+    <!-- android -->
+    <platform name="android">
+
+        <config-file target="AndroidManifest.xml" parent="/manifest/application">
+            <activity android:name="com.google.zxing.client.android.CaptureActivity" android:screenOrientation="landscape" android:clearTaskOnLaunch="true" android:configChanges="orientation|keyboardHidden" android:theme="@android:style/Theme.NoTitleBar.Fullscreen" android:windowSoftInputMode="stateAlwaysHidden" android:exported="false">
+                <intent-filter>
+                    <action android:name="com.phonegap.plugins.barcodescanner.SCAN"/>
+                    <category android:name="android.intent.category.DEFAULT"/>
+                </intent-filter>
+            </activity>
+        </config-file>
+
+    </platform>
+
+    <platform name="windows8">
+        <js-module src="src/windows8/BarcodeScannerProxy.js" name="BarcodeScannerProxy">
+            <merges target=""/>
+        </js-module>
+        <config-file target="package.appxmanifest" parent="/Package/Capabilities">
+            <DeviceCapability Name="webcam"/>
+        </config-file>
+        <framework src="src/windows8/lib/ZXing.winmd" custom="true"/>
+        <framework src="src/windows8/lib/WinRTBarcodeReader.winmd" custom="true"/>
+    </platform>
+
+    <!-- Windows Phone 8 -->
+    <platform name="wp8">
+
+        <config-file target="config.xml" parent="/*">
+            <feature name="BarcodeScanner">
+                <param name="wp-package" value="BarcodeScanner"/>
+            </feature>
+        </config-file>
+
+        <config-file target="Properties/WMAppManifest.xml" parent="/Deployment/App/Capabilities">
+            <Capability Name="ID_CAP_ISV_CAMERA"/>
+        </config-file>
+
+        <framework src="src/wp8/lib/zxing.wp8.0.dll" custom="true"/>
+
+        <source-file src="src/wp8/BarcodeScanner.cs"/>
+        <source-file src="src/wp8/BarcodeScannerTask.cs"/>
+        <source-file src="src/wp8/BarcodeScannerUI.xaml"/>
+        <source-file src="src/wp8/BarcodeScannerUI.xaml.cs"/>
+
+    </platform>
+</plugin>