Minor fixes

joanasesinando · Feb 10, 2021 · 6edb60c · 6edb60c
1 parent 8244939
commit 6edb60c
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 12 deletions.
diff --git a/src/app/_services/fenix/fenix.service.ts b/src/app/_services/fenix/fenix.service.ts
@@ -23,8 +23,6 @@ export class FenixService {
 
  constructor(public translateService: TranslateService, public errorService: ErrorService) { }
 
- // FIXME: degree in course??
-
  /********************** CORRECT STRUCTURE **********************/
 
  parseDegree(degreeJson): Degree {

diff --git a/src/app/_util/testing.service.ts b/src/app/_util/testing.service.ts
@@ -27,13 +27,6 @@ export class TestingService {
 
  constructor() { }
 
- /* --------------------------------------------------------------------------------
- * Creates a Course focusing only on its shifts.
- * -------------------------------------------------------------------------------- */
- createShiftsOnlyCourse(shifts: Shift[]): Course { // FIXME: if not used remove
- return new Course(this.COURSE_ID, this.COURSE_NAME, this.COURSE_ACRONYM, this.COURSE_SEMESTER, undefined, undefined, shifts);
- }
-
  /* --------------------------------------------------------------------------------
  * Creates a Shift focusing only on time it occurs.
  * --------------------------------------------------------------------------------

diff --git a/src/app/homepage/homepage.component.spec.ts b/src/app/homepage/homepage.component.spec.ts
@@ -362,7 +362,6 @@ describe('HomepageComponent', () => {
  });
 
  describe('Preparing courses to generate schedules for', () => {
- // TODO: patch A / B more tests
  let course: Course;
 
  beforeEach(() => {

diff --git a/src/app/homepage/homepage.component.ts b/src/app/homepage/homepage.component.ts
@@ -194,7 +194,7 @@ export class HomepageComponent implements OnInit, AfterViewInit {
  this.typesOfClassesPicked.clear();
 
  // Clean selected courses
- for (const course of this.selectedCourses) { // FIXME: no need?
+ for (const course of this.selectedCourses) {
  this.removeCourse(course.id);
  }
 
@@ -227,7 +227,7 @@ export class HomepageComponent implements OnInit, AfterViewInit {
  this.loadCoursesBasicInfo(academicTerm, degreeID);
 
  // Reset selected courses state
- for (const course of this.stateService.selectedCourses) { // FIXME: no need?
+ for (const course of this.stateService.selectedCourses) {
  this.addCourse(course.id);
  }