ignore access for completed enrollments on course user page

test plan:
* have a course that restricts students from viewing in past
* have a student
* view the course user page for the student
* conclude the enrollment
* refresh the page
* it should say the enrollment was completed (not inactive)

closes #CNVS-33646

Change-Id: Ib7f0c58f4ffd71748a62315a1c6abd86795727fd
Reviewed-on: https://gerrit.instructure.com/102478
Tested-by: Jenkins
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
QA-Review: Deepeeca Soundarrajan <dsoundarrajan@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>

Author: maneframe

app/views/context/roster_user.html.erb

@@ -129,7 +129,7 @@
                   </div>
                 <% end %>
                 <div style="font-size: 0.8em; padding-left: 20px; <%= hidden unless enrollment.completed? || enrollment.inactive? %>" class="completed_at_holder">
-                  <% if enrollment.inactive? %>
+                  <% if enrollment.workflow_state == 'inactive' %>
                     <%= t 'inactive' %>
                   <% elsif enrollment.completed_at %>
                     <%= t 'completed_enrollment', "completed *%{completed_at}*", :completed_at => datetime_string(enrollment.completed_at), :wrapper => '<span class="completed_at">\1</span>' %>