Document "data snapshot" aspect/intent of AbstractDbData and subclasses more explicitly
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- java
- Área
- documentation
Línea de trabajo
Comienza con la documentación de Concepts / Elements of the Database y el JavaDoc de AbstractDbData y sus subclases. Haz explícitos el propósito de la instantánea de datos y la indicación de crear una nueva Table después de las actualizaciones de la base de datos; se considerará terminado cuando sea poco probable que los lectores interpreten Table como un objeto de datos recargable.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Dear AssertJ-DB team, Dear Joel,
I found one of my friends wrapping Table to allow for a "data reload feature". As I understand the JavaDoc of that class, the intended purpose of this class and the sibling AbstractDbData subclasses is to hold a snapshot of DB data and provide a fluent assertion API for this data snapshot.
The web site http://joel-costigliola.github.io/assertj/assertj-db-concepts.html states for e. g. Table and Request, that these objects represent a "Table in" or a "Request on" the database.
This probably misled my friend (who has a decent background in RDBMS) into thinking that a Table object could / should also be used in more complex query/update scenarios, whereas its original purpose is "only" to contain the data snapshot used for upcoming fluent assertions.
Would it be possible to make this intention of holding a data snapshot from different sources (Table, Request) more explicit in the "Concepts" / "Elements of the Database" page, maybe a "Caution" paragraph elaborating a bit on this? In particular, I consider a hint like "If you need to re-load data after DB update operations, please create a new Table instance" very useful.
Looking forward to your feedback. Thanks in advance & keep up the good work. Very much appreciated.
- Lenguaje dominante
- Java
- Estrellas
- 130
- Forks
- 21
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de assertj/assertj-db
-
Add toString() and equals() implementations for core domain types (Table, Row, Value, Column) Abierto
assertj/assertj-db#324 · 1 comentario · 1 asignado ·
-
question
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
assertj/assertj-db#292 · 1 comentario ·
-
row(index).column("ABC") and row().row().row().column("ABC") always referring to row 0 itself Abiertoquestion
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
assertj/assertj-db#290 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
assertj/assertj-db#259 · 1 comentario ·
-
Support R2DBC connection Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
assertj/assertj-db#206 ·
Todos los issues de assertj/assertj-db
Issues similares
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Abiertoarea/plugin
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
kestra-io/plugin-kestra#190 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Add canonical URLs and a sitemap Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
apache/rocketmq-dashboard#5064 ·