Jkmarx/fix perms api error

[jkmarx]

Fix bug related to misunderstanding of perms, share vs add perms
@scottx611x I did confirm via locally and unit tests the perms change fixes the issue.

Resolves #2957

[codecov]

Codecov Report

Merging #2959 into release-1.6.6 will decrease coverage by 0.18%.
The diff coverage is 98.48%.

@@                Coverage Diff                @@
##           release-1.6.6    #2959      +/-   ##
=================================================
- Coverage          59.58%   59.39%   -0.19%     
=================================================
  Files                433      433              
  Lines              27736    27197     -539     
  Branches            1274     1274              
=================================================
- Hits               16527    16155     -372     
+ Misses             11209    11042     -167

Impacted Files	Coverage Δ
refinery/core/api.py	38.43% <0%> (-0.46%)	⬇️
refinery/core/test_views.py	100% <100%> (ø)	⬆️
refinery/core/views.py	42.94% <100%> (ø)	⬆️
refinery/data_set_manager/utils.py	74.77% <0%> (-0.18%)	⬇️
refinery/tool_manager/tests.py	99.82% <0%> (-0.01%)	⬇️
refinery/data_set_manager/tests.py	99.28% <0%> (ø)	⬆️
refinery/data_set_manager/search_indexes.py	94.44% <0%> (+2.37%)	⬆️

---

Continue to review full report at Codecov.

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

[jkmarx]

add_%s perms is used for checking ownership on non-shareable resources like analyses and workflows
data_set uses share_%s (only owners can share data_sets right now)