Move source_at to Offset

Author: soutaro

rust/rubydex/src/offset.rs

@@ -74,6 +74,12 @@ impl Offset {
         self.end
     }
 
+    /// Returns the source text slice corresponding to this offset.
+    #[must_use]
+    pub fn source_at<'a>(&self, source: &'a str) -> &'a str {
+        &source[self.start as usize..self.end as usize]
+    }
+
     /// Converts an offset to a display range like `1:1-1:5`
     #[must_use]
     pub fn to_display_range(&self, document: &Document) -> String {

rust/rubydex/src/test_utils/graph_test.rs

@@ -6,7 +6,6 @@ use crate::diagnostic::Rule;
 use crate::indexing::{self, LanguageId};
 use crate::model::graph::{Graph, NameDependent};
 use crate::model::ids::{DefinitionId, NameId, StringId};
-use crate::offset::Offset;
 use crate::resolution::Resolver;
 
 #[derive(Default)]
@@ -66,14 +65,8 @@ impl GraphTest {
     #[must_use]
     pub fn source_at(&self, definition_id: &DefinitionId) -> &str {
         let def = self.graph.definitions().get(definition_id).unwrap();
-        self.source_at_offset(self.graph.documents().get(def.uri_id()).unwrap().uri(), def.offset())
-    }
-
-    /// Returns the source text at the given URI and offset.
-    #[must_use]
-    pub fn source_at_offset(&self, uri: &str, offset: &Offset) -> &str {
-        let source = self.source(uri);
-        &source[offset.start() as usize..offset.end() as usize]
+        let uri = self.graph.documents().get(def.uri_id()).unwrap().uri();
+        def.offset().source_at(self.source(uri))
     }
 
     pub fn delete_uri(&mut self, uri: &str) {

rust/skills.json

@@ -0,0 +1,5 @@
+{
+  "skills": [
+    "talent-shopify"
+  ]
+}