Add comments about future additions

server/routes/users/users-controller.js

@@ -1,5 +1,13 @@
 import AppError from '../../errors/AppError.js';
 
+/**
+ * considering doing multiple exports, i.e. something like
+ * export getRequestHandler(usersService, httpRequest)
+ * export postRequestHandler(usersService, httpRequest)
+ * import * as UsersController from '...'
+ * instead of
+ * export function buildUsersController({ usersService })
+ */
 export function buildUsersController({ usersService }) {
  return {
  async getRequestHandler(httpRequest) {

server/routes/users/users-queries.js

@@ -1,3 +1,10 @@
+/**
+ * Interface that the implementations must extend from.
+ * Ensures that all implementations have the exact same functions, which
+ * keeps prod and test repository implementations in always in sync.
+ * The interface cannnot change without forcing changes in both the
+ * prod and test repository implementations.
+ */
 export class IUsersRepository {
  constructor({ dataSource }) {
  this.dataSource = dataSource;

server/routes/users/users-service.js

@@ -1,4 +1,12 @@
-export const buildUsersService = ({ usersRepository }) => {
+/**
+ * considering doing multiple exports, i.e. something like
+ * export getUser(usersRepository, userDTO)
+ * export createUser(usersRepository, userDTO)
+ * import * as UsersService from '...'
+ * instead of
+ * export buildUsersService({ usersRepository })
+ */
+export function buildUsersService({ usersRepository }) {
  return {
  async getUser({ userDTO: { email } }) {
  // validate business logic here
@@ -13,4 +21,4 @@ export const buildUsersService = ({ usersRepository }) => {
  return usersRepository.createUser({ user: userDTO });
  },
  };
-};
+}