fix: remove folder when conversation list is empty [WPB-15892]

[Garzas]

WPB-15892 [Android] It should not be possible to create empty folders

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Because backend doesn't clean folders by itself we need to check if they are empty after conversation from folder removal

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Test Results

3 515 tests  +2   3 407 ✅ +2   6m 29s ⏱️ +34s
  603 suites ±0     108 💤 ±0 
  603 files   ±0       0 ❌ ±0 

Results for commit b2270e3. ± Comparison against base commit 20791fd.

♻️ This comment has been updated with latest results.

[github-actions]

Bencher Report

Branch	fix/remove-empty-folder
Testbed	ubuntu-latest

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

• Latency (nanoseconds (ns))

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results

Benchmark	Latency	microseconds (µs)
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInFiles	📈 view plot ⚠️ NO THRESHOLD	696.94
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory	📈 view plot ⚠️ NO THRESHOLD	349,882.92
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark	📈 view plot ⚠️ NO THRESHOLD	1,341,356.07
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark	📈 view plot ⚠️ NO THRESHOLD	22,404.44

🐰 View full continuous benchmarking report in Bencher

[borichellow]

I believe task is not only about removing folder after removing the last conversation from it. But also about creating an empty folder in "move to folder" flow. And this PR looks like doesn't solve it. 🤔

[borichellow]

I believe task is not only about removing folder after removing the last conversation from it. But also about creating an empty folder in "move to folder" flow. And this PR looks like doesn't solve it. 🤔

nvm, found Android PR for that :)

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.84%. Comparing base (20791fd) to head (b2270e3).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3285      +/-   ##
===========================================
- Coverage    50.84%   50.84%   -0.01%     
===========================================
  Files         1607     1607              
  Lines        58052    58056       +4     
  Branches      5203     5204       +1     
===========================================
+ Hits         29514    29516       +2     
- Misses       26521    26524       +3     
+ Partials      2017     2016       -1     

see 5 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 20791fd...b2270e3. Read the comment docs.

[datadog-wireapp]

Datadog Report

Branch report: fix/remove-empty-folder
Commit report: 95c7c82
Test service: kalium-jvm

✅ 0 Failed, 3407 Passed, 108 Skipped, 1m 0.59s Total Time