danger-android_permissions_checker 0.0.1 → 0.0.2
Sign up to get free protection for your applications and to get access to all the features.
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 26d593c45170d3f8e0d46bde541d8cefc81e5596c026cd58d6c1cf0c9254f930
|
4
|
+
data.tar.gz: a9f90f3673ba04bb47e51d3fdff462991fc0b375571520141ba4108764f8856a
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 5803cd6d6f55c62f8767d8cf08e1ccc89f3344bce3b4228dd32b8e36bc8f34f923b5982a300a45252b68aaed156d19c49e42cd96adbb01a850a805d8b348a410
|
7
|
+
data.tar.gz: ef06d442a6631b0f73646e14874f4ad18b4743aa5df45aeb66ea27f210aeab56e8f3efb694b170c083804cc78ea5522c474b6e945488f73a08286e4e26881fe7
|
data/.travis.yml
CHANGED
data/README.md
CHANGED
@@ -12,10 +12,10 @@ Check permissions between current permissions and apk generated on CI service. i
|
|
12
12
|
|
13
13
|
## Usage
|
14
14
|
|
15
|
-
Create permission list file of your apk using aapt, like this.
|
15
|
+
Create current permission list file of your apk using aapt, like this.
|
16
16
|
|
17
17
|
```bash
|
18
|
-
$ aapt d permissions /path/to/
|
18
|
+
$ aapt d permissions /path/to/apk > permissions.txt
|
19
19
|
```
|
20
20
|
|
21
21
|
Add this to Dangerfile. Specify apk generated by CI service and permission list file generated by above command.
|
@@ -23,7 +23,7 @@ Add this to Dangerfile. Specify apk generated by CI service and permission list
|
|
23
23
|
```
|
24
24
|
android_permissions_checker.check(
|
25
25
|
apk: '/path/to/generated_apk_by_CI',
|
26
|
-
permission_list_file: /path/to/
|
26
|
+
permission_list_file: /path/to/permissions.txt
|
27
27
|
)
|
28
28
|
```
|
29
29
|
|
@@ -1,5 +1,3 @@
|
|
1
|
-
require 'fileutils'
|
2
|
-
|
3
1
|
module Danger
|
4
2
|
class DangerAndroidPermissionsChecker < Plugin
|
5
3
|
def check(apk: nil, permission_list_file: nil)
|
@@ -21,7 +19,7 @@ module Danger
|
|
21
19
|
deleted = current_permissions - generated_permissions
|
22
20
|
added = generated_permissions - current_permissions
|
23
21
|
message = ""
|
24
|
-
|
22
|
+
|
25
23
|
if deleted.length > 0
|
26
24
|
message += "### Deleted permissions\n"
|
27
25
|
deleted.each do |v|
|
@@ -40,7 +38,7 @@ module Danger
|
|
40
38
|
|
41
39
|
unless message.empty?
|
42
40
|
markdown(message)
|
43
|
-
warn(
|
41
|
+
warn("APK permissions changed, see below. Should update `#{permission_list_file}` if it is intended change.")
|
44
42
|
end
|
45
43
|
end
|
46
44
|
end
|
@@ -4,7 +4,7 @@ module Danger
|
|
4
4
|
describe Danger::DangerAndroidPermissionsChecker do
|
5
5
|
let(:dangerfile) { testing_dangerfile }
|
6
6
|
let(:plugin) { dangerfile.android_permissions_checker }
|
7
|
-
|
7
|
+
|
8
8
|
it "should be a plugin" do
|
9
9
|
expect(Danger::DangerAndroidPermissionsChecker.new(nil)).to be_a Danger::Plugin
|
10
10
|
end
|
@@ -27,7 +27,7 @@ module Danger
|
|
27
27
|
end
|
28
28
|
|
29
29
|
before do
|
30
|
-
allow(current_permission_file).to receive(:nil?).and_return(false)
|
30
|
+
allow(current_permission_file).to receive(:nil?).and_return(false)
|
31
31
|
allow(apk).to receive(:nil?).and_return(false)
|
32
32
|
allow(File).to receive(:exist?).with(current_permission_file).and_return(true)
|
33
33
|
allow(File).to receive(:exist?).with(apk).and_return(true)
|
@@ -49,7 +49,7 @@ module Danger
|
|
49
49
|
it do
|
50
50
|
plugin.check(apk: apk, permission_list_file: current_permission_file)
|
51
51
|
expect(dangerfile.status_report[:warnings].length).to eq(1)
|
52
|
-
expect(dangerfile.status_report[:warnings][0]).to eq(
|
52
|
+
expect(dangerfile.status_report[:warnings][0]).to eq("APK permissions changed, see below. Should update `#{current_permission_file}` if it is intended change.")
|
53
53
|
expect(dangerfile.status_report[:markdowns][0].message).not_to include('Deleted')
|
54
54
|
expect(dangerfile.status_report[:markdowns][0].message).to include('Added')
|
55
55
|
|
@@ -64,7 +64,7 @@ module Danger
|
|
64
64
|
it do
|
65
65
|
plugin.check(apk: apk, permission_list_file: current_permission_file)
|
66
66
|
expect(dangerfile.status_report[:warnings].length).to eq(1)
|
67
|
-
expect(dangerfile.status_report[:warnings][0]).to
|
67
|
+
expect(dangerfile.status_report[:warnings][0]).to include('APK permissions changed, see below.')
|
68
68
|
expect(dangerfile.status_report[:markdowns][0].message).to include('Deleted')
|
69
69
|
expect(dangerfile.status_report[:markdowns][0].message).not_to include('Added')
|
70
70
|
end
|
@@ -78,7 +78,7 @@ module Danger
|
|
78
78
|
it do
|
79
79
|
plugin.check(apk: apk, permission_list_file: current_permission_file)
|
80
80
|
expect(dangerfile.status_report[:warnings].length).to eq(1)
|
81
|
-
expect(dangerfile.status_report[:warnings][0]).to
|
81
|
+
expect(dangerfile.status_report[:warnings][0]).to include('APK permissions changed, see below.')
|
82
82
|
expect(dangerfile.status_report[:markdowns][0].message).to include('Deleted')
|
83
83
|
expect(dangerfile.status_report[:markdowns][0].message).to include('Added')
|
84
84
|
end
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: danger-android_permissions_checker
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.0.
|
4
|
+
version: 0.0.2
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- mataku
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2018-
|
11
|
+
date: 2018-07-26 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: danger-plugin-api
|
@@ -122,7 +122,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
122
122
|
version: '0'
|
123
123
|
requirements: []
|
124
124
|
rubyforge_project:
|
125
|
-
rubygems_version: 2.7.
|
125
|
+
rubygems_version: 2.7.7
|
126
126
|
signing_key:
|
127
127
|
specification_version: 4
|
128
128
|
summary: A Danger plugin to check diff of android apk permissions.
|