glpi-plugin icon indicating copy to clipboard operation
glpi-plugin copied to clipboard

WIP F-Droid market

Open btry opened this issue 6 years ago • 12 comments

rebase of #193

provides a way to get apps from fdroid, download them as new packages and update them automatically.

Changes description

Checklist

Please check if your PR fulfills the following specifications:

  • [x] Tests for the changes have been added
  • [ ] Docs have been added/updated

Estimated time

Assignee :tomato:
@btry 2630
@ajsb85 1
@DIOHz0r 19

References

Closes #N/A Related #N/A Depends on #N/A

btry avatar Aug 21 '18 12:08 btry

@DIOHz0r this feature started in october 2017... sorry I don't remember with good accuracy the time needed to do this feature.

btry avatar Aug 21 '18 13:08 btry

There were the following issues with this Pull Request

  • Commit: 575a7d253dfa03995f0fbdef80cede31baf98a12
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

ghost avatar Aug 22 '18 14:08 ghost

There were the following issues with this Pull Request

  • Commit: f26134b8450ea095349ae6c1008e89f824d93ec8
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

ghost avatar Aug 22 '18 14:08 ghost

I solved all merge conflicts which will occur with release/2.0.0, except one. We must wait the merge of the release branch, then rebase this one, solve the conflict, then merge into develop.

Checking the build remains greek for now.

btry avatar Aug 22 '18 14:08 btry

There were the following issues with this Pull Request

  • Commit: bb330991aeed5e6d779e27debb42a0242a3decbd
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

ghost avatar Aug 22 '18 14:08 ghost

There were the following issues with this Pull Request

  • Commit: 771d03e768e696cf068d19e79803a26fa31480a9
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

ghost avatar Aug 23 '18 09:08 ghost

There were the following issues with this Pull Request

  • Commit: f306c372f2ef0d067b01ece54f631d6ba846d38a
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

ghost avatar Aug 24 '18 16:08 ghost

wait ! i'm still working on it !

btry avatar Sep 05 '18 15:09 btry

this error happens randomly, the unit test is quite simple, don't see yet why the failing tests gets 2 or 0 instead of expected 1...

Failure (25 tests, 174/174 methods, 0 void method, 0 skipped method, 0 uncompleted method, 1 failure, 0 error, 0 exception)!
failures
> There is 1 failure:
=> tests\units\PluginFlyvemdmFDroidMarket::testUpdateRepository():
In file /home/travis/build/flyve-mdm/glpi/plugins/flyvemdm/tests/suite-unit/PluginFlyvemdmFdroidMarket.php on line 164, integer() failed: []
-Expected
+Actual
@@ -2 +2 @@
-int(1)
+int(0)
> Total tests duration: 28.95 seconds.
> Total tests memory usage: 271.93 Mb.
> Running duration: 59.24 seconds.
Failure (25 tests, 174/174 methods, 0 void method, 0 skipped method, 0 uncompleted method, 1 failure, 0 error, 0 exception)!
failures
> There is 1 failure:
=> tests\units\PluginFlyvemdmFDroidMarket::testUpdateRepository():
In file /home/travis/build/flyve-mdm/glpi/plugins/flyvemdm/tests/suite-unit/PluginFlyvemdmFdroidMarket.php on line 164, integer() failed: []
-Expected
+Actual
@@ -2 +2 @@
-int(1)
+int(0)
Coverage report is available at file:///home/travis/build/flyve-mdm/glpi/plugins/flyvemdm/development/coverage/develop/index.html

btry avatar Sep 07 '18 14:09 btry

It looks like there are some missing files because this error in CI...

file_get_contents(/home/travis/build/flyve-mdm/glpi/plugins/flyvemdm/tests/suite-unit/../fixtures/fdroid-repo-app-removed.xml): failed to open stream: No such file or directory

DIOHz0r avatar Sep 07 '18 17:09 DIOHz0r

Hello, I did a rebase here for fixing the conflicts.

DIOHz0r avatar Sep 25 '18 22:09 DIOHz0r

I still need to validate that updates of apps in fdroid propagate automatically to devices when needed

btry avatar Sep 26 '18 06:09 btry