Improve test coverage

[gorkemarslan]

• I try to improve the unit test coverage discussed in Improve unit test coverage #952. Coverage increases from 54% to 65%.
• I fix the errors that I detect in the backend.
  For example:
  When we want to get draft threads we get published ones, not draft ones.
  Notice how is_draft parameters are used in these functions:
  

  OpenCiviWiki/project/accounts/api.py

  Lines 124 to 135 in 201368a

  	@action(detail=True)
  	def drafts(self, request, user__username=None):
  	"""
  	Gets the draft threads of the selected account
  	/accounts/{username}/drafts
  	"""
  	user = get_user_model().objects.get(username=user__username)
  	draft_threads = Thread.objects.filter(author=user, is_draft=False)
  	serializer = ThreadSerializer(
  	draft_threads, many=True, context={"request": request}
  	)
  	return Response(serializer.data)

  
  and vice versa
  

  OpenCiviWiki/project/accounts/api.py

  Lines 111 to 122 in 201368a

  	@action(detail=True)
  	def threads(self, request, user__username=None):
  	"""
  	Gets the preferred categories of the selected account
  	/accounts/{username}/categories
  	"""
  	user = get_user_model().objects.get(username=user__username)
  	draft_threads = Thread.objects.filter(author=user).exclude(is_draft=False)
  	serializer = ThreadSerializer(
  	draft_threads, many=True, context={"request": request}
  	)
  	return Response(serializer.data)

Another example:

OpenCiviWiki/project/threads/api.py

Lines 68 to 70 in 201368a

	problems = []
	for idx, item in enumerate(c_data):
	problems[idx]["score"] = c_scores[idx]

Since problems is an empty array, it is not appropriate to add elements by inserting them via index. The correct way is to use append() method.

• I add api_view(['GET']) decorators to get_user(), get_profile(), and get_card() functions to give them DRF permissions defined in settings.py. Thus AnonymousUser cannot get sensitive user data.

• Interestingly, GitHub shows my changes as removing files and recreating them. 🤔