Fix meteor/audit-argument-checks eslint rule

Author: mauriciovieira

feedback/collection/server/publications.js

@@ -1,14 +1,21 @@
+import { check } from 'meteor/check';
 import { Meteor } from 'meteor/meteor';
-// Collection imports
+
 import FeedbackVotes from '/feedback_votes/collection';
 import Feedback from '../';
 
 Meteor.publish('apiBackendFeedback', (apiBackendId) => {
+  // Make sure apiBackendId is a String
+  check(apiBackendId, String);
+
   // show feedback to specific API
   return Feedback.find({ apiBackendId });
 });
 
 Meteor.publish('getAllVotesForSingleFeedback', (feedbackId) => {
+  // Make sure apiBackendId is a String
+  check(feedbackId, String);
+
   // show feedbackvotes for single feedback
   return FeedbackVotes.find({ feedbackId });
 });

feedback/server/methods.js

@@ -1,17 +1,28 @@
+import { check } from 'meteor/check';
 import { Meteor } from 'meteor/meteor';
 import { TAPi18n } from 'meteor/tap:i18n';
-// Collection imports
+
 import FeedbackVotes from '/feedback_votes/collection';
 import Feedback from '../collection';
 
 Meteor.methods({
   deleteFeedback (feedbackItemId) {
+    // Make sure feedbackItemId is a String
+    check(feedbackItemId, String);
+
     // 1. Remove feedback votes
     FeedbackVotes.remove({ feedbackId: feedbackItemId });
+
     // 2. Remove feedback item
     Feedback.remove(feedbackItemId);
   },
   submitVote (feedbackId, vote) {
+    // Make sure feedbackId is a String
+    check(feedbackId, String);
+
+    // Make sure vote is a Number
+    check(vote, Number);
+
     // Get current user ID
     const userId = Meteor.userId();