feat(null): upsert and onConflict. Fix bad syntax in queryParams For …

…upserts

Issues :
- #3 Upsert does not really upsert
- #8 add on_conflict for upsert

querydsl-postgrest-resttemplate-adapter/src/main/java/fr/ouestfrance/querydsl/postgrest/PostgrestRestTemplate.java

@@ -131,5 +131,4 @@ private URI getUri(String resource, Map<String, List<String>> params) {
         return uriBuilder.build().encode().toUri();
     }
 
-
 }

querydsl-postgrest/src/main/java/fr/ouestfrance/querydsl/postgrest/PostgrestRepository.java

@@ -133,11 +133,11 @@ public BulkResponse<T> upsert(List<Object> values, BulkOptions options) {
      * @return map of query params for on conflict if annotation OnConflict is present otherwise empty map
      */
     private Map<String, List<String>> getUpsertQueryParams() {
-        return Optional.ofNullable(this.getClass().getAnnotation(OnConflict.class))
-                .map(OnConflict::columnNames)
-                .map(Arrays::asList)
-                .map(onConflictList-> Map.of(ON_CONFLICT_QUERY_PARAMS, onConflictList))
-                .orElse(Map.of());
+        OnConflict onConflict = this.getClass().getAnnotation(OnConflict.class);
+        if (Objects.nonNull(onConflict)) {
+            return Map.of(ON_CONFLICT_QUERY_PARAMS, List.of(String.join(",", onConflict.columnNames())));
+        }
+        return Collections.emptyMap();
     }
 
     /**